Closed
Bug 1561216
Opened 5 years ago
Closed 5 years ago
Change `StaticPrefs` from a class to a namespace
Categories
(Core :: Preferences: Backend, enhancement)
Core
Preferences: Backend
Tracking
()
RESOLVED
FIXED
mozilla69
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
Attachments
(2 files)
This will allow the static prefs to be spread across multiple files.
|
Assignee | |
Comment 1•5 years ago
|
||
This doesn't change the way C++ code uses static prefs. But it does slightly
change how Rust code uses static prefs, specifically the name generated by
bindgen is slightly different.
The commit also improves some comments.
Depends on D35763
|
|
Assignee | |
Comment 2•5 years ago
|
||
Pushed by nnethercote@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/7f8969072f73 Remove unneeded `mozilla::` qualifiers in Preferences.cpp. r=froydnj https://hg.mozilla.org/integration/mozilla-inbound/rev/f6e047701cf4 Change `StaticPrefs` from a class to a namespace. r=froydnj
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/7f8969072f73
https://hg.mozilla.org/mozilla-central/rev/f6e047701cf4
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox69:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
|
||
Updated•5 years ago
|
Assignee: nobody → n.nethercote
You need to log in
before you can comment on or make changes to this bug.
Description
•