Update of patch #5937 (project freeciv):
Status: Done => In Progress
Open/Closed: Closed => Open
_______________________________________________________
Follow-up Comment #2:
Sorry for not reviewing this patch in detail before now, but after thinking
about this I feel the need to discuss the array support in the JSON protocol.
As a start, please take a look at the array definition in JSON at
http://json.org/ - the definition of a JSON array is: An array is an ordered
collection of values. An array begins with [ (left bracket) and ends with ]
(right bracket). Values are separated by , (comma).
In this array implementation, arrays are implemented as separate key-value
pairs. So each array element is a separate key-value pair. For example, part
of the tech object will look like this in the new array implementation:
{name:"Robotics", "req_0":52, "req_1":17}
I think that arrays in the json protocol should be implemented in the
"json-way" as described on json.org. The tech object will then look like
this:
{name:"Robotics", "reqs":[52,17]}
The native json array format is significantly more compact, and much easier to
iterate.
At the moment most of the dialogs in Freeciv-web are not working, since they
rely on json arrays the native way.
So let's work some more on improving the arrays support in the json-protocol
and Freeciv-web.
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?5937>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
