Closed Bug 1561216 Opened 5 months ago Closed 5 months ago

Change `StaticPrefs` from a class to a namespace

Categories

(Core :: Preferences: Backend, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: njn, Assigned: njn)

References

Details

Attachments

(2 files)

This will allow the static prefs to be spread across multiple files.

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

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
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Assignee: nobody → n.nethercote
Blocks: 1563139
No longer blocks: 1560530
You need to log in before you can comment on or make changes to this bug.