Closed
Bug 197
Opened 27 years ago
Closed 26 years ago
warning fix : ns/lib/layout/edtbuf.c
Categories
(MozillaClassic Graveyard :: Composer, defect, P3)
Tracking
(Not tracked)
VERIFIED
WONTFIX
M12
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.
Updated•26 years ago
|
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → WONTFIX
Comment 1•26 years ago
|
||
old bug, old code base. Marking won't fix. Please re-open if I am in error.
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 2•25 years ago
|
||
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.
Description
•