Closed
Bug 1490161
Opened 6 years ago
Closed 5 years ago
StaticPrefs names violates Gecko C++ naming conventions
Categories
(Core :: Preferences: Backend, enhancement, P3)
Core
Preferences: Backend
Tracking
()
RESOLVED
WONTFIX
| Tracking | Status | |
|---|---|---|
| firefox64 | --- | affected |
People
(Reporter: MatsPalmgren_bugz, Unassigned)
Details
Bug 1448222 comment 11 summarizes some pro/cons for snake_case vs CamelCase.
I think this one:
> - It's contrary to usual Mozilla style.
trumps all other by far and that we should change all these names
to use CamelCase.
I think naming convention exceptions like this are harmful and it's
not worth the benefit of being able to grep the code using the same
pattern as for the pref string.
|
||
Comment 1•5 years ago
|
||
We now generate all the static prefs from a YAML file, and all the usages of CamelCase prefs have been converted to snake_case in bugs like bug 1562331. I think this bug becomes WONTFIX ...
Status: NEW → RESOLVED
Closed: 5 years ago
Component: General → Preferences: Backend
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•