🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: comp.windows.interviews
Article #759

Re: Bug and fix in ListUpdater 3.1b3

#759
From: Mark Linton
Date: Fri, 16 Oct 1992 17:23
18 lines
1175 bytes
In article <ldpbsmINNm57@news.bbn.com>, jpalmucc@bbn.com (Jeff Palmucci) writes:
|> I think I have found a bug in the implementation of ListUpdater.
|> 
|> In the file list.h:
|> 
|> the line
|> 
|> inline void ListUpdater(List)::remove_cur() { list_->remove(cur_); } \
|> 
|> should be
|> 
|> inline void ListUpdater(List)::remove_cur() { end_--; list_->remove(cur_); } \
|> 
|> If I am wrong, could someone in the know tell me. I have patched my local
|> copy and to not want to inadvertently break other things.

A simpler fix, which also handles a related problem in ListItr, is to change
references to end_ in the iterators to call list_->count().
Technical Details
Message-ID: <1bmtq9INN61i@fido.asd.sgi.com>
Path: arc1-ca.pugleaf.net!sparky!uunet!kithrup!hoptoad!pacbell.com!sgiblab!sgigate!odin!fido!marktwain.rad.sgi.com!linton
References: <ldpbsmINNm57@news.bbn.com>

Show Full Headers
Complete Article Headers
Path: arc1-ca.pugleaf.net!sparky!uunet!kithrup!hoptoad!pacbell.com!sgiblab!sgigate!odin!fido!marktwain.rad.sgi.com!linton
From: linton@marktwain.rad.sgi.com (Mark Linton)
Newsgroups: comp.windows.interviews
Subject: Re: Bug and fix in ListUpdater 3.1b3
Date: 16 Oct 1992 17:23:21 GMT
Organization: sgi
Lines: 19
Distribution: world
Message-ID: <1bmtq9INN61i@fido.asd.sgi.com>
References: <ldpbsmINNm57@news.bbn.com>
Reply-To: linton@marktwain.rad.sgi.com (Mark Linton)
NNTP-Posting-Host: marktwain.rad.sgi.com