A tiny clean-up.

Comment on attachment 8920924 [details] [diff] [review] (part 1) - Remove unused PrefTypeFlags values Review of attachment 8920924 [details] [diff] [review]: ----------------------------------------------------------------- ::: modules/libpref/Preferences.cpp @@ -244,5 @@ > PREF_FLAG_LOCKED = 4, > PREF_FLAG_USERSET = 8, > - PREF_FLAG_CONFIG = 16, > - PREF_FLAG_REMOTE = 32, > - PREF_FLAG_LILOCAL = 64, FWIW, they were effectively removed in bug 99611.

Comment on attachment 8920933 [details] [diff] [review] (part 3) - Remove nsPrefBranch::RemoveObserverFromMap declaration Review of attachment 8920933 [details] [diff] [review]: ----------------------------------------------------------------- Static analysis for this kind of stuff could be interesting (declared methods that we never define).

https://hg.mozilla.org/integration/mozilla-inbound/rev/69b08044cafb8a2cf2e2426382084bbd9279137f Bug 1410765 (part 1) - Remove unused PrefTypeFlags values. r=glandium. https://hg.mozilla.org/integration/mozilla-inbound/rev/37eb7a2fb6ac6c897f8bccbf0e103a3e2b40159c Bug 1410765 (part 2) - Redo clang-format on libpref/. r=glandium. https://hg.mozilla.org/integration/mozilla-inbound/rev/71a01e882f44c3143b9b2dd777f988f58e7af862 Bug 1410765 (part 3) - Remove nsPrefBranch::RemoveObserverFromMap declaration. r=glandium.

> Static analysis for this kind of stuff could be interesting (declared > methods that we never define). Sylvestre, do you know if a clang static analysis could be written to detect methods that are declared but never defined? (Like the one removed by part 3 above.)

(See comment 8)

https://hg.mozilla.org/mozilla-central/rev/69b08044cafb https://hg.mozilla.org/mozilla-central/rev/37eb7a2fb6ac https://hg.mozilla.org/mozilla-central/rev/71a01e882f44

(In reply to Nicholas Nethercote [:njn] from comment #8) > > Static analysis for this kind of stuff could be interesting (declared > > methods that we never define). > > Sylvestre, do you know if a clang static analysis could be written to detect > methods that are declared but never defined? (Like the one removed by part 3 > above.) This would have several issues: 1. Possibility of external addressing of functions, that is resolved only during linkage. So in a compilation unit we would only define functions that would be externally called by other compilation units. 2. The impossibility of clang to match decl and call simultaneously, in an ast tree, this would have to be done only by transvering manually the tree by matching for each declaration all of the funcCalls and then triage them by signature. 3. There is the possibility to also call functions by pointer to functions or pointers to member function that again will complicate the resolution and would generate a lot of false positives. Having all of this issues I don't see the real gaining of having this kind of checker, only the fact that we might reduce TXT size.

Fair enough!

I asked Andi to answered about that :) Merci Andi!