If I'm not mistaken this patch allocates N*sizeof(_Tp) bytes of storage and 
deallocates N bytes when sized deallocation is enabled?

Shouldn't __return_temporary_buffer deallocate N*sizeof(_Tp) instead to match 
the value passed to new?

________________________________
De : [email protected] <[email protected]> de la part de 
François Dumont <[email protected]>
Envoyé : jeudi 18 juillet 2019 07:41
À : [email protected] <[email protected]>; gcc-patches 
<[email protected]>
Objet : sized delete in _Temporary_buffer<>

As we adopted the sized deallocation in the new_allocator why not doing
the same in _Temporary_buffer<>.

     * include/bits/stl_tempbuf.h (__detail::__return_temporary_buffer):
New.
     (~_Temporary_buffer()): Use latter.
     (_Temporary_buffer(_FIterator, size_type)): Likewise.

Tested w/o activating sized deallocation. I'll try to run tests with
this option activated.

Ok to commit ?

François

Reply via email to