Closed Bug 306 Opened 27 years ago Closed 26 years ago

explict type cast needed in edtele.cpp lines 5955 & 10668

Categories

(MozillaClassic Graveyard :: Composer, defect, P3)

1998-04-08
DEC
All
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: berkley, Assigned: cmanske)

Details

Created by Berkley Shands (berkley@cs.wustl.edu) on Wednesday, May 6, 1998 9:33:58 AM PDT Additional Details : from mozilla/19980429/mozilla/lib/layout: cxx: Error: edtele.cpp, line 5955: no instance of constructor "CEditEndContainerElement::CEditEndContainerElement" matches the argument list argument types are: (long) CEditEndContainerElement* pEndContainer = new CEditEndContainerElement(NULL); -------------------------------------------------------------------------------^ cxx: Error: edtele.cpp, line 10668: no instance of constructor "CEditContainerElement::CEditContainerElement" matches the argument list argument types are: (CEditElement *, long, int, ED_Alignment) CEditContainerElement(pParent, NULL, 0 /* Never used for an end container */, ED_ALIGN_LEFT) ------------------------------^ the fixes - CEditEndContainerElement::CEditEndContainerElement(CEditElement *pParent) : CEditContainerElement(pParent, (PA_Tag*) NULL, 0 /* Never used for an end container */, ED_ALIGN_LEFT) and CEditEndContainerElement* pEndContainer = new CEditEndContainerElement((CEditElement *)NULL); Geez, I need to figure out you patch system... berkley
Assignee: djw → brade
Component: Layout → Composer
Assignee: brade → cmanske
Status: NEW → ASSIGNED
What compiler are you using? We don't have any trouble with any compilers we use. The 2 versions of the constructor have different number of arguments: CEditContainerElement::CEditContainerElement( CEditElement *pParent, PA_Tag *pTag, int16 csid, ED_Alignment defaultAlign ): and CEditContainerElement::CEditContainerElement( IStreamIn *pStreamIn, CEditBuffer *pBuffer): so it seems the compiler should be able to resolve the types, but it won't hurt to put in the explicit castes, so this fix is OK.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Checked in 10/29/98
Status: RESOLVED → VERIFIED
OS: All
Marking verified against old code base
You need to log in before you can comment on or make changes to this bug.