[Issue 3415] broken JSON output

[via Digitalmars-d-bugs]

https://issues.dlang.org/show_bug.cgi?id=3415

Walter Bright  changed:

   What|Removed |Added

Version|1.050   |D1

--

[Issue 3415] broken JSON output

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=3415


Don  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||clugd...@yahoo.com.au
 Resolution||FIXED


--- Comment #3 from Don  2010-05-05 19:06:53 PDT ---
Fixed DMD2.044

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---

[Issue 3415] broken JSON output

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=3415


Rainer Schuetze  changed:

   What|Removed |Added

   Keywords||patch
 CC||r.sagita...@gmx.de


--- Comment #2 from Rainer Schuetze  2010-04-14 00:04:47 
PDT ---
This happens for declarations in an AttribDeclaration, where the code for
adding commas is missing. Here's the patch:

Index: json.c
===
--- json.c(revision 432)
+++ json.c(working copy)
@@ -214,11 +214,17 @@

 if (d)
 {
+size_t offset = buf->offset;
 for (unsigned i = 0; i < d->dim; i++)
 {   Dsymbol *s = (Dsymbol *)d->data[i];
+if (offset != buf->offset)
+{   buf->writestring(",\n");
+offset = buf->offset;
+}
 //printf("AttribDeclaration::toJsonBuffer %s\n", s->toChars());
 s->toJsonBuffer(buf);
 }
+JsonRemoveComma(buf);
 }
 }

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---

[Issue 3415] broken JSON output

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=3415


Ary Borenszweig  changed:

   What|Removed |Added

 CC||echocham...@gmail.com


--- Comment #1 from Ary Borenszweig  2009-10-26 01:01:09 
PDT ---
*** Issue 3440 has been marked as a duplicate of this issue. ***

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---