Closed Bug 818218 Opened 12 years ago Closed 12 years ago

Refactor and clean ExtendedJSONObject parsing code.

Categories

(Firefox for Android Graveyard :: Android Sync, defect)

Product:
Firefox for Android Graveyard
Component:
Android Sync
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla20

People

(Reporter: nalexander, Assigned: nalexander)

References

Details

This will reduce memory pressure a little.  Bug 747721 is an error at org.json.simple.parser.Yylex.<init>(Yylex.java:231); that line is allocating a 16kb buffer for input text.  We allocate one of those for each JSONParser instance we create.  Let's make this bug be a simple attempt to create less JSONParser instances.
Reworking this to be a first step towards improving our parsing code.  This will just be cleaning and groundwork for the next improvements.
Summary: Make ExtendedJSONObject class use a single static JSONParser instance. → Refactor and clean ExtendedJSONObject parsing code.
Assignee: nobody → nalexander
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/e2df102b0fe7
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Blocks: 802723
Product: Mozilla Services → Android Background Services
Product: Android Background Services → Firefox for Android
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.