Closed
Bug 915855
Opened 12 years ago
Closed 12 years ago
Change "const static" to "static const"
Categories
(Core :: JavaScript Engine, enhancement)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla26
People
(Reporter: sunfish, Assigned: sunfish)
Details
Attachments
(1 file)
|
16.08 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
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 1•12 years ago
|
||
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+
|
Assignee | |
Comment 2•12 years ago
|
||
|
|
||
Comment 3•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
|
||
Updated•12 years ago
|
Assignee: general → sunfish
You need to log in
before you can comment on or make changes to this bug.
Description
•