Closed Bug 305 Opened 26 years ago Closed 26 years ago

edtbuf.cpp SelectTable overload needs explicit type cast

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:24:53 AM PDT Additional Details : cxx: Error: edtbuf.cpp, line 14446: no instance of overloaded function "CEditBuffer::SelectTable" matches the argument list argument types are: (int, long, CEditTableElement *) SelectTable(FALSE, NULL, (CEditTableElement*)pElement); ----------------^ 1 error detected in the compilation of "edtbuf.cpp". make: *** [OSF1V4.0_DBG.OBJ/edtbuf.o] Error 1 mozilla/lib/layout> this is in the 19980429 edition. if( pElement->IsTable() ) { SelectTable(FALSE, (LO_TableStruct*) NULL, (CEditTableElement*)pElement); return; } else if( pElement->IsTableCell() ) { SelectCell(FALSE, (LO_CellStruct*) NULL, (CEditTableCellElement*)pElement); return; } berkley
Assignee: djw → brade
Component: Layout → Composer
Ahhh, edtbuf.cpp, old friend. How we love to port thee...
Assignee: brade → cmanske
reassign to cmanske
Status: NEW → ASSIGNED
Casting a NULL seems ludicrous. Does it work if you use zero (0 or 0L) instead?
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Checked in 10/29/98
OS: All
QA Contact: 4015
qa contact set to gerardok@netscape.com for review in case this should be checked against new code base
Charlie, please help us verify this bug by giving us a test case or mark it yourself as verified.
Status: RESOLVED → VERIFIED
There is no possible test case - its just a change to allow compiling on certain UNIX compilers. Its all moot anyway, since none of this code will be used in Seamonkey 5.0
You need to log in before you can comment on or make changes to this bug.