Recent changes to bugs

Space (#20) does not need conversion

Nils Achterholt — Sun, 19 Jun 2011 08:21:56 -0000

In class function TJSONObject.quote(s: string): string; Space (#20) gets converted to "\u0020", which is unnecessary, because JSON allows spaces.
Adding ' ' to the NoConversion-array results in a much better human-readable string.

Incorrect toString for _double

Nils Achterholt — Mon, 30 May 2011 10:57:15 -0000

The function toString:string of _Double returns the value in quotationmarks, which is not standard JSON.
It also loses precision, since formatFloat is used with a formatstring which has only 2 digits after the decimal point.

Solution:
function _Double.toString: string;
begin
result := StringReplace(FloatToStr(fvalue),',','.',[rfReplaceAll]);
end;

Memory Leak

Yi Zhang — Thu, 12 Nov 2009 14:54:31 -0000

function TJSONObject.toString(indentFactor, indent: integer): string;
This function has a line: _keys := keys(); The _keys not free until the function exit.

Please add a try .. finally _keys.Free end; for this function.

The line: key := x.nextValue().toString(); will cause a memory leak, too.
Please write in this way:

with x.nextValue() do
begin
key := toString();
Free; //Fix memory leak. By creation_zy
end;

escaped utf strings doesn't work

Anonymous — Wed, 14 Jun 2006 09:07:10 -0000

I've got response from server which looks like

{"string":"\u043f\u043e\u043f\u0440\u043e\u0431\u0443\u0439
\u0437\u0430\u0439\u0434\u0438 \u043f\u043e\u0434
\u043b\u043e\u0433\u0438"}

and such \uXXXX strings returns to me like uXXXX

I had found a problem (required code was commented and
was not right) and now problem resolved

Where can i put a patch?