Closed Bug 512637 Opened 15 years ago Closed 15 years ago

Use newlines instead of JSON collection for incremental parsing

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

Attachments

(1 file)

Bug 485573 added some complicated incremental JSON parsing code... we can get rid of it and use application/newlines!
Depends on: 507433
Attached patch v1Splinter Review
Yay!
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #396664 - Flags: review?(thunder)
Comment on attachment 396664 [details] [diff] [review]
v1

r=thunder

this patch is awesome!
Attachment #396664 - Flags: review?(thunder) → review+
http://hg.mozilla.org/labs/weave/rev/f4edc187f9be
Switch to newline mode when using a collection record handler, and look for newlines! Easy! Update test to provide newline-separated strings instead of JSON.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.6
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: