1621341 - Remove flags parameter from ReportError APIs

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P1)

Description

[Tom S. (please needinfo tschuster)]

I think there is only very few places after bug 1620583, that don't pass JSREPORT_ERROR to the various ReportErrorXXX APIs. Mostly just JSREPORT_WARNING, which should use a separate API, so that we can further split error and warning reporting.

Comment 1

[Tom S. (please needinfo tschuster)]

Attachment: Bug 1621341 - Replace ReportValueErrorFlags with ReportValueError. r?tcampbell

Comment 2

[Tom S. (please needinfo tschuster)]

Attachment: Bug 1621341 - Replace JS_ReportErrorFlagsXXX calls using JSREPORT_ERROR. r?tcampbell

Comment 3

[Tom S. (please needinfo tschuster)]

Attachment: Bug 1621341 - Remove JS_ReportErrorFlags and introduce new internal WarnNumber API. r?tcampbell

Depends on D66326

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/48c37fc6c779
Replace ReportValueErrorFlags with ReportValueError. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/17e828c47f0e
Replace JS_ReportErrorFlagsXXX calls using JSREPORT_ERROR. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/d4c4c027ba53
Remove JS_ReportErrorFlags and introduce new internal WarnNumber API. r=tcampbell

Comment 5

[Tom S. (please needinfo tschuster)]

Attachment: Bug 1621341 - Remove flags from ReportCompileWarning. r?arai

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/48c37fc6c779
https://hg.mozilla.org/mozilla-central/rev/17e828c47f0e
https://hg.mozilla.org/mozilla-central/rev/d4c4c027ba53

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4d8d735099bc
Remove flags from ReportCompileWarning. r=arai

Comment 8

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/4d8d735099bc