Re: [ cplusplus-bug 580197 ] misc.h: ~iterator() delete token

[David Sugar]

Yes, we should always use a matching delete[] if originally new xxx[] was 
used, even for base types like char arrays.  On some c++ compilers, []'s have 
extra block information, and so using a simple delete will cause a memory 
leak.

On Tuesday 13 August 2002 16:20, Federico Montesino Pouzols wrote:
>   Hi, this is now fixed in the cvs hosted at savannah.
>
>
> ------------------------------------------------------
>  virtual ~iterator() { if (token) *token='\0'; delete
> token; }
>
> should be delete [] token because token is allocated as
> an array in keydata.cpp. A check of keydata.cpp will
> show a correct call to delete[] token in operator++.
>
> Probably won't cause a problem because elements of
> array are primitive type "char", but a check of the ARM
> says behavior is implementation dependent.
>
> Bug is in 1.9.7 and 2.0.99
>
>
>
>
> _______________________________________________
> Bug-commoncpp mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/bug-commoncpp