Closed Bug 336988 Opened 14 years ago Closed 12 years ago

Useless null check of aPos in nsTextFrame::PeekOffset

Categories

(Core :: Layout: Text and Fonts, defect, trivial)

x86
Linux
defect
Not set
trivial

Tracking

()

RESOLVED WONTFIX

People

(Reporter: timeless, Assigned: timeless)

References

(Blocks 1 open bug, )

Details

(Keywords: coverity)

Attachments

(1 file)

 
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #221194 - Flags: superreview?
Attachment #221194 - Flags: review?(smontagu)
Attachment #221194 - Flags: superreview? → superreview?(dbaron)
Attachment #221194 - Flags: review?(smontagu) → review+
Comment on attachment 221194 [details] [diff] [review]
look before leaping

sr=dbaron, but not that roc has a copy of nsTextFrame that may also have this code that he should be landing soon
Attachment #221194 - Flags: superreview?(dbaron) → superreview+
I believe this patch is obsolete...
Flags: blocking1.9?
Obseleted by new textframe
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
Flags: blocking1.9?
You need to log in before you can comment on or make changes to this bug.