_IMPL_NS_HTML is #defined in many places, but has no effect

RESOLVED FIXED

Status

SeaMonkey
Build Config
RESOLVED FIXED
16 years ago
13 years ago

People

(Reporter: Brian Ryner (not reading), Assigned: timeless)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

40.45 KB, patch
hacker formerly known as seawood@netscape.com
: review+
Brian Ryner (not reading)
: superreview+
Details | Diff | Splinter Review
(Assignee)

Comment 1

16 years ago
Created attachment 69208 [details] [diff] [review]
zap

-DEBUG_rhp doesn't do what's intended, so this fixes that too.
(Reporter)

Comment 2

16 years ago
for that matter, DEBUG_rhp doesn't do anything either in those directories.
might as well remove it.

r=bryner
(Assignee)

Comment 3

16 years ago
ok.

These also appear to have no consumers of the following:
_IMPL_NS_CHARDET _IMPL_NS_INTL _IMPL_NS_LAYOUT _IMPL_NS_LOCALE _IMPL_NS_LWBRK 
_IMPL_NS_MOZBROWSER _IMPL_NS_STRRES _IMPL_NS_UCONV _IMPL_NS_UI 
_IMPL_NS_UNICHARUTIL _IMPL_NS_WEB

Comment 4

16 years ago
timeless, can you update the patch if needed & get sr=/a=?

Comment 5

16 years ago
Created attachment 93674 [details] [diff] [review]
patch v1.1

patch includes changes to htmlparser.xml, but doesn't include cvs removal of
htmlparserSharedPrefix_debug.h & htmlparserSharedPrefix.h which aren't needed
anymore.
Attachment #69208 - Attachment is obsolete: true
(Reporter)

Comment 6

16 years ago
Comment on attachment 93674 [details] [diff] [review]
patch v1.1

r/sr=bryner.  If the prefix files aren't needed any more, please go ahead and
remove them as well.
Attachment #93674 - Flags: superreview+
Patch has been checked in & htmlparserSharedPrefix{,_debug}.h have been removed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.