702551 - copy over JSON doc annotations from FR mainline

Status: RESOLVED WONTFIX

(Tamarin Graveyard :: Documentation, defect)

Description

[Felix S. Klock II [:pnkfelix, :fklock]]

There are documentation comments in JSON.as that should be propagated over to tamarin-redux.

This is a work item that will ease integrations from TR/TR-sec to FRmain.

(Don't let this block integrations unless you cannot stand to have diff's outstanding between FRmain and TR-sec.  As in: I'm not going to let it block integration right now.)

Comment 1

[Felix S. Klock II [:pnkfelix, :fklock]]

not gonna spend time on this right now

Comment 2

[Brent Baker]

I think this is pretty much done, minus a single blank line inserted into one of the API documentation blocks.

Comment 3

[Brent Baker]

The majority of this bug was taken care of in CL@1036353, again the only diff at the moment between Mainline and tamarin is a single blank line in documentation block for the stringify() API.


==== //depot/main/FlashRuntime/Main/code/third_party/avmplus/core/JSON.as#10 (text) - //depot/users/FlashRuntime/avm/code/third_party/avmplus/core/JSON.as#4 (text) ==== content
@@ -202,6 +202,7 @@
         * is a non-negative number <i>x</i>, then the gap is <i>x</i> space characters, to a maximum of ten spaces.
         * When the argument passed to the <code>space</code> parameter is not a String a Number, it is converted to a
         * String for use as the gap in the output. Execution then proceeds.</p>
+        *
         * <p>When the <code>stringify()</code> method encounters a cyclic structure, it throws a TypeError with error ID <code>kJSONCyclicStructure</code>. It also posts a localized error message.</p>
 
         * @param value The ActionScript value to be converted into a JSON string

Comment 4

[Sylvestre Ledru [:Sylvestre]]

Tamarin is a dead project now. Mass WONTFIX.

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Tamarin isn't maintained anymore. WONTFIX remaining bugs.