Prevent increment order bug on some compilers.

dewitt-garmin · dewitt-garmin · commit f056f2c053df · 2014-09-16T20:43:23.000-05:00
Visual studio interprets post decrements very literally and increments
the child's length_iterated element instead of the array's element.

Change-Id: I06abcebc6c33587a9e3d1339da8298bfd5830b4f
Type: Bug Fix
diff --git a/json-builder.c b/json-builder.c
@@ -783,7 +783,8 @@ void json_serialize_ex (json_char * buf, json_value * value, json_serialize_opts
                PRINT_NEWLINE();
             }
 
-            value = value->u.array.values [((json_builder_value *) value)->length_iterated ++];
+            ((json_builder_value *) value)->length_iterated++;
+            value = value->u.array.values [((json_builder_value *) value)->length_iterated - 1];
             continue;
 
          case json_object:

Original file line number	Diff line number	Diff line change
`@@ -783,7 +783,8 @@ void json_serialize_ex (json_char * buf, json_value * value, json_serialize_opts`
`783`	`783`	`PRINT_NEWLINE();`
`784`	`784`	`}`
`785`	`785`
`786`		`- value = value->u.array.values [((json_builder_value *) value)->length_iterated ++];`
	`786`	`+ ((json_builder_value *) value)->length_iterated++;`
	`787`	`+ value = value->u.array.values [((json_builder_value *) value)->length_iterated - 1];`
`787`	`788`	`continue;`
`788`	`789`
`789`	`790`	`case json_object:`