Closed Bug 1312956 Opened 8 years ago Closed 7 years ago

Remove JSEXN_NONE from documentation comment in js.msg

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox52 --- wontfix
firefox54 --- fixed

People

(Reporter: arai, Assigned: arai)

Details

Attachments

(1 file)

JSEXN_NONE was removed in bug 1274922, but it's still in the documentation comment in js.msg.
it should be removed or replaced with other one.
removed JSEXN_NONE, and updated description for EXCEPTION_NAME.
Assignee: nobody → arai.unmht
Status: NEW → ASSIGNED
Attachment #8836211 - Flags: review?(evilpies)
Comment on attachment 8836211 [details] [diff] [review]
Update js.msg documentation comment.

Review of attachment 8836211 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/js.msg
@@ +18,5 @@
>   *
>   * <ARGUMENT_COUNT> is an integer literal specifying the total number of
>   * replaceable arguments in the following format string.
>   *
> + * <EXCEPTION_NAME> is JSExnType enum value, defined in jsapi.h.

I think "is an"
Attachment #8836211 - Flags: review?(evilpies) → review+
https://hg.mozilla.org/mozilla-central/rev/a15a4bd9f8e4
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: