Closed Bug 915855 Opened 6 years ago Closed 6 years ago

Change "const static" to "static const"

Categories

(Core :: JavaScript Engine, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: sunfish, Assigned: sunfish)

Details

Attachments

(1 file)

This changes places in the code that use "const static" to use "static const" instead. There's no functional difference, but since const is part of the type, and static is not, it's cleaner to put the const second.
Attachment #803976 - Flags: review?(terrence)
Comment on attachment 803976 [details] [diff] [review]
const-static.patch

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

r=me

I've never really thought about it, but in my head the type-qualifier has always come before the storage-class-specifiers. No idea why. I'll try to remember in future to flip the order so that I am consistent with our now-consistent style.
Attachment #803976 - Flags: review?(terrence) → review+
https://hg.mozilla.org/mozilla-central/rev/a84f8558ca61
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
Assignee: general → sunfish
You need to log in before you can comment on or make changes to this bug.