Consolidate JSON serialization code

Bug #1029292 reported by Chris Hillery
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zorba
Incomplete
Medium
Ghislain Fourny

Bug Description

Paul has recently consolidated some of the JSON module serialization code, in particular adding a single function for escaping JSON strings (zorba::json::serialize(ostream&, char const*)).

We should ensure we are using this code in all places that it makes sense, most specifically in the JSONiq serialization code (src/api/serialization/serializer.cpp, class json_emitter).

If there are other avenues for code sharing (eg., for JSON objects and arrays), they should be taken as well.

Tags: jsoniq
Chris Hillery (ceejatec)
Changed in zorba:
importance: Undecided → Medium
assignee: nobody → Paul J. Lucas (paul-lucas)
Revision history for this message
Paul J. Lucas (paul-lucas) wrote :

I already did make the code in src/api/serialization/serializer.cpp share code. I'm not aware of any other serialization code. Hence, reassigning.

Changed in zorba:
status: New → Incomplete
assignee: Paul J. Lucas (paul-lucas) → Ghislain Fourny (gislenius)
Till Westmann (tillw)
tags: added: 1029292
tags: added: jsoniq
removed: 1029292
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.