Closed Bug 434748 Opened 12 years ago Closed 12 years ago

Make mailnews/base/search compile with frozen linkage

Categories

(MailNews Core :: Search, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

Attached patch The fixSplinter Review
Following on from bug 407295 and others, there's just a few more changes we need in mailnews/base/search to get it compiling.
Attachment #321768 - Flags: superreview?(bienvenu)
Attachment #321768 - Flags: review?(bienvenu)
Comment on attachment 321768 [details] [diff] [review]
The fix

r/sr=bienvenu, but just out of curiousity, why do we need ctype.h? Not because we changed IsUTF8 to MsgIsUTF8...
Attachment #321768 - Flags: superreview?(bienvenu)
Attachment #321768 - Flags: superreview+
Attachment #321768 - Flags: review?(bienvenu)
Attachment #321768 - Flags: review+
(In reply to comment #1)
> (From update of attachment 321768 [details] [diff] [review])
> r/sr=bienvenu, but just out of curiousity, why do we need ctype.h? Not because
> we changed IsUTF8 to MsgIsUTF8...
> 
I responded on IRC. Basically they aren't related and its just an extra part of the tidy up required after bug 407925.

Patch checked in -> fixed.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.