1759754 - Broken "use the static preference API" link in "C++ Coding style" documentation

Status: RESOLVED FIXED

(Core :: Preferences: Backend, defect)

Description

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

I was reading through the "C++ Coding style" guide in the source tree documentation. In this section, I read:

You should use the static preference API for working with preferences.

However, the "static preference API" link is broken. We should remove it or link to the new location of the docs.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:KrisWright, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Janvi Bajoria [:janvi01]]

Hey, I would like to work on this.
Just a quick question, should I remove the link? or Should I link to the new location? (Please provide the new location)
Thanks!

Comment 3

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

I don't know – :KrisWright, do you?

Comment 4

[Kris Wright :KrisWright]

(In reply to Janvi Bajoria [:janvi01] from comment #2)

Hey, I would like to work on this.
Just a quick question, should I remove the link? or Should I link to the new location? (Please provide the new location)
Thanks!

The new libpref documentation can be found here:
https://firefox-source-docs.mozilla.org/modules/libpref/index.html

I think we could replace the broken link with this new link.

Comment 5

[Janvi Bajoria [:janvi01]]

Attachment: Bug 1759754 - fix use the static preference API link in C++ coding style documentation r?KrisWright

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:janvi01, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/12370cbe605f
fix use the static preference API link in C++ coding style documentation r=KrisWright

Comment 8

[bszekely]

https://hg.mozilla.org/mozilla-central/rev/12370cbe605f