Closed Bug 434748 Opened 16 years ago Closed 16 years ago

Make mailnews/base/search compile with frozen linkage

Categories

(MailNews Core :: Search, defect)

Product:
MailNews Core
Component:
Search
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla1.9

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

The fix
2.14 KB, patch
Bienvenu
: review+
Bienvenu
: superreview+
Details | Diff | Splinter Review
Attached patch The fix — — Splinter 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: 16 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.