Closed
Bug 388093
Opened 18 years ago
Closed 18 years ago
CNavDTD misuses PRBool as an integer
Categories
(Core :: DOM: HTML Parser, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha8
People
(Reporter: mrbkap, Assigned: mrbkap)
Details
Attachments
(1 file)
|
2.94 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
There are a couple of places where CNavDTD uses a PRBool with value -1 to mean "unknown state." taras's tool in bug 266048 tried to deal with them, but failed, since it simply coerces the -1 assignments to booleans, which isn't what we want. I'm filing this bug to fix those cases manually.
| Assignee | ||
Comment 1•18 years ago
|
||
This is fairly straightforward, I think.
Attachment #272255 -
Flags: superreview?(jst)
Attachment #272255 -
Flags: review?(jst)
Updated•18 years ago
|
Attachment #272255 -
Flags: superreview?(jst)
Attachment #272255 -
Flags: superreview+
Attachment #272255 -
Flags: review?(jst)
Attachment #272255 -
Flags: review+
| Assignee | ||
Comment 2•18 years ago
|
||
Fix checked into trunk.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•