Unused variable warning in GetContentMap

RESOLVED FIXED in mozilla5

Status

()

defect
RESOLVED FIXED
8 years ago
9 months ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla5
x86
Linux
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [build_warning])

Attachments

(1 attachment)

Posted patch Patch v1Splinter Review
No description provided.
Attachment #515488 - Flags: review?(ben)
Flags: in-testsuite-
First, I definitely appreciate cleaning up warnings in the optimized builds! Thanks for taking the time.

However, I'm worried that there a lot of places equivalent to this, and this solution is not as readable. Perhaps there is a more elegant solution? Chris, do you have any patterns for this kind of thing?
Blassey, see the decisions at the bottom of bug 577899.  It looks like people generally use ifdefs everywhere for this situation.
Comment on attachment 515488 [details] [diff] [review]
Patch v1

(In reply to comment #2)
> Blassey, see the decisions at the bottom of bug 577899.  It looks like people
> generally use ifdefs everywhere for this situation.

Blassey?

Thanks for the link. I'm happy if this is the norm, but please indent the Init() call.
Attachment #515488 - Flags: review?(ben) → review+
Assignee

Updated

8 years ago
Attachment #515488 - Flags: approval2.0?
Assignee

Updated

8 years ago
Whiteboard: [build_warning][needs review] → [build_warning][needs approval]

Updated

8 years ago
Attachment #515488 - Flags: approval2.0? → approval2.0-
Assignee

Updated

8 years ago
Depends on: post2.0
Whiteboard: [build_warning][needs approval] → [build_warning]

Comment 5

8 years ago
http://hg.mozilla.org/mozilla-central/rev/962e067eaa13
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2

Updated

9 months ago
Product: Core → Core Graveyard
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.