Closed Bug 1372877 Opened 7 years ago Closed 7 years ago

Use nullptr in /js (clang-tidy: modernize-use-nullptr)

Categories

(Core :: JavaScript Engine, enhancement)

47 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: janx, Assigned: janx)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

In /js there are several opportunities to use `nullptr` instead of `0` or `NULL`.

These can be automatically identified and acted on by the clang-tidy static analyzer (rule "modernize-use-nullptr").
This patch was generated with the following command:

run-clang-tidy-4.0.py -p obj-x86_64-pc-linux-gnu/ -fix -checks=-*,modernize-use-nullptr js/
Attachment #8877550 - Flags: review+
Added "r=nbp", r+ carried over.
Attachment #8877606 - Flags: review+
Attachment #8877550 - Attachment is obsolete: true
R+ and try looks green (apart from 2 unrelated-looking oranges), so please land this patch.
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e1386053622e
Use nullptr in /js (clang-tidy: modernize-use-nullptr). r=nbp
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/e1386053622e
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: