Closed Bug 197 Opened 27 years ago Closed 26 years ago

warning fix : ns/lib/layout/edtbuf.c

Categories

(MozillaClassic Graveyard :: Composer, defect, P3)

1998-03-31
defect

Tracking

(Not tracked)

VERIFIED WONTFIX

People

(Reporter: rick, Assigned: Brade)

Details

Created by Rick Osborne (rick@rixsoft.com) on Monday, April 13, 1998 6:10:03 PM PDT Additional Details : (In the continuing quest to eliminate even compiler warnings ... ) (14345) : warning : ''formatting'' : unreferenced local variable (14345) : warning : ''i'' : unreferenced local variable (14451) : warning : ''i'' : unreferenced local variable Basic stuff. We didn''t get an warning for selection, but that''s just because it''s an object instead of a primitive. We don''t use it anyway. (Look ma, I saved an object creation/deletion!) --- edtbuf.old.cpp Wed Apr 08 17:52:23 1998 +++ edtbuf.cpp Mon Apr 13 21:10:17 1998 @@ -14320,7 +14320,9 @@ CEditBuffer::ReplaceOnce( char *pReplace you want replaced has been selected already. */ +#ifdef FORMATING_FIXED /* these vars use only in this context */ CEditSelection selection; int i; EDT_CharacterData *formatting; /* This will hold the formatting of the selected text */ +#endif #ifdef FORMATING_FIXED @@ -14352,5 +14354,6 @@ CEditBuffer::ReplaceLoop(char *pReplaceT XP_Bool bBackward, XP_Bool bDoWrap ) { - int i; + /* We don''t need this, but I''m sure it was a good idea at the time */ + /* int i; */ LO_Element *start_ele_loc, *end_ele_loc, *original_start_ele_loc, *original_end_ele_loc; Updated by (brade@netscape.com) on Thursday, April 16, 1998 8:45:03 PM PDT Additional Details : Reassign bug to myself since I'm the owner of this file.
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → WONTFIX
old bug, old code base. Marking won't fix. Please re-open if I am in error.
Status: RESOLVED → VERIFIED
setting to an approximate milestone so it can be off of the no TFV list
Target Milestone: --- → M12
You need to log in before you can comment on or make changes to this bug.