Remove unused MOZ_PLACES_BOOKMARKS define

VERIFIED FIXED in mozilla5

Status

()

Toolkit
Places
VERIFIED FIXED
9 years ago
7 years ago

People

(Reporter: mak, Assigned: emorley)

Tracking

Trunk
mozilla5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

the only global switch for Places is MOZ_PLACES, MOZ_PLACES_BOOKMARKS is completely unused, and we can remove it.
(Assignee)

Comment 1

7 years ago
Given this is build config, should I be basing off of m-c instead of http://hg.mozilla.org/projects/places/ ?
OS: Windows Vista → All
Hardware: x86 → All
(In reply to comment #1)
> Given this is build config, should I be basing off of m-c instead of
> http://hg.mozilla.org/projects/places/ ?
mozilla-central, please.  We haven't merged m-c into it in a while (although I'll probably do that now).
(Assignee)

Comment 3

7 years ago
Created attachment 525031 [details] [diff] [review]
Patch v1 (based on m-c rev 09b605eb3e0d)

Great - have based it off of m-c tip.
Assignee: nobody → bmo
Status: NEW → ASSIGNED
Attachment #525031 - Flags: review?(sdwilsh)
(Assignee)

Updated

7 years ago
Summary: remove unused MOZ_PLACES_BOOKMARKS define → Remove unused MOZ_PLACES_BOOKMARKS define
Comment on attachment 525031 [details] [diff] [review]
Patch v1 (based on m-c rev 09b605eb3e0d)

thanks!
Attachment #525031 - Flags: review?(sdwilsh) → review+
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

7 years ago
Just spotted this was pushed as:
http://hg.mozilla.org/mozilla-central/rev/d114ab94a859
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
(Assignee)

Comment 6

7 years ago
http://mxr.mozilla.org/mozilla-central/search?string=MOZ_PLACES_BOOKMARKS
...returns 0 results.

-> Verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.