Closed Bug 414551 Opened 18 years ago Closed 17 years ago

gCallbacksEnabled in prefapi.cpp is unused and should be killed

Categories

(Core :: Preferences: Backend, defect)

Product:
Core
Component:
Preferences: Backend
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9.2a1

People

(Reporter: asqueella, Assigned: Swatinem)

References

(
URL
)

Details

Attachments

(1 file, 1 obsolete file)

updated [Checkin: Comment 3]
1.83 KB, patch
benjamin
: review+
benjamin
: superreview+
Details | Diff | Splinter Review
Whiteboard: [good first bug]
Attached patch remove the var (obsolete) — — Splinter Review
Small patch that does the job.
Assignee: nobody → arpad.borsos
Status: NEW → ASSIGNED
Attachment #335498 - Flags: review?(darin.moz)
I didn't notice this has interfered with my profilesharing patch. Updated patch.
Attachment #335498 - Attachment is obsolete: true
Attachment #351531 - Flags: superreview?(benjamin)
Attachment #351531 - Flags: review?(benjamin)
Attachment #335498 - Flags: review?(darin.moz)
Attachment #351531 - Flags: superreview?(benjamin)
Attachment #351531 - Flags: superreview+
Attachment #351531 - Flags: review?(benjamin)
Attachment #351531 - Flags: review+
Keywords: checkin-needed
Comment on attachment 351531 [details] [diff] [review] updated [Checkin: Comment 3] http://hg.mozilla.org/mozilla-central/rev/407791718438
Attachment #351531 - Attachment description: updated → updated [Checkin: Comment 3]
(Fwiw, I'd like to see this into 1.9.1 too.)
URL: http://mxr.mozilla.org/seamonkey/sear... → http://mxr.mozilla.org/mozilla-centra...
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [good first bug]
Target Milestone: --- → mozilla1.9.2a1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: