Closed Bug 1230 Opened 26 years ago Closed 26 years ago

Possible extra const in layout/base/public/nsIFrame.h

Categories

(Core :: Layout, defect, P2)

Product:
Core
Component:
Layout
Platform:
Sun
Solaris
Type:
defect

Tracking

()

Status:
VERIFIED INVALID

People

(Reporter: tor, Assigned: peterl-retired)

Details

I believe that there may be an extra "const" in the following function declaration: --- nsIFrame.h-orig Sat Oct 31 00:53:23 1998 +++ nsIFrame.h Sat Oct 31 00:53:36 1998 @@ -169,7 +169,7 @@ /** * Get the style data associated with this frame. */ - NS_IMETHOD GetStyleData(nsStyleStructID aSID, const nsStyleStruct*& aStyleStruct) const = 0; + NS_IMETHOD GetStyleData(nsStyleStructID aSID, nsStyleStruct*& aStyleStruct) const = 0; /** * Accessor functions for geometric and content parent.
Assignee: kipp → peterl
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → INVALID
Nope. The method is correctly defined as const since it doesn't and can't modify the frame. The out param nsStyleStruct is also correctly defined as const since it is an immutable structure after the style resolution process (which is over before the frame gets it). Note that there are mutable pathways into the nsStyleStructs, but this isn't one of them.
Status: RESOLVED → VERIFIED
Marking Verified/Invalid
You need to log in before you can comment on or make changes to this bug.