Closed Bug 615256 Opened 14 years ago Closed 14 years ago

warning: 'PRBool FramesOnSameLineHaveZeroHeight(nsIFrame*)' defined but not used

Categories

(Core :: Layout, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: timeless, Assigned: timeless)

Details

Attachments

(1 file)

(deleted), patch
MatsPalmgren_bugz
: review+
dbaron
: approval2.0+
Details | Diff | Splinter Review
layout/base/nsCaret.cpp:150: warning: ‘PRBool FramesOnSameLineHaveZeroHeight(nsIFrame*)’ defined but not used
Attached patch remove it (deleted) — Splinter Review
this merely removes dead code and thus is safe for 2.0 http://mxr.mozilla.org/mozilla-central/search?string=FramesOnSameLineHaveZeroHeight Found one matching line /layout/base/nsCaret.cpp (View Hg log or Hg annotations) * line 150 -- FramesOnSameLineHaveZeroHeight(nsIFrame* aFrame) Found one matching line
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #493717 - Flags: review?(matspal)
Attachment #493717 - Flags: approval2.0?
Comment on attachment 493717 [details] [diff] [review] remove it r=mats
Attachment #493717 - Flags: review?(matspal) → review+
Attachment #493717 - Flags: approval2.0? → approval2.0+
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: