Closed Bug 232111 Opened 16 years ago Closed 11 years ago
Wallet should use the newer ns
IPref Service APIs instead of ns IPref
nsIPref is deprecated
Summary: Remove nsIPref from wallet → Wallet should use the newer nsIPrefService APIs instead of nsIPref
I'm a bit of a noob here. This seems like a hopefully fairly simple bug.
Jesse: I'm not sure this is worth it ... AFAIK, the wallet module is not used by Firefox or Thunderbird, only Seamonkey. Eventually the plan is to move Seamonkey to Toolkit, so the entire wallet extension will be obsolete ... But no, this is not a simple bug. The Wallet code is horrible and ugly - there are far better bugs for noobs to take on. Take a look at <http://www.mozilla.org/contribute/hacking/first-bugs/>.
I'm just posting this here for the sake of trying to get a clean tree. I do not think I'll fixup wallet any more than this (but maybe), so if dveditz or someone wants to review this, that'd be cool.
(In reply to comment #3) > Created an attachment (id=195723) [details] > cleans up nsWalletService.cpp > > I'm just posting this here for the sake of trying to get a clean tree. > > I do not think I'll fixup wallet any more than this (but maybe), so if dveditz > or someone wants to review this, that'd be cool. Did you mean to ask for a review on this?
R.WontFix, after bug 473827.
You need to log in before you can comment on or make changes to this bug.