diff options
author | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2016-11-23 19:49:21 +0100 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2016-11-24 05:09:05 +0000 |
commit | b367c81d06f64e500063ebb814eda5e8d55309ca (patch) | |
tree | befb401f63253018cd8accb5902d314101c4db78 /basic/source/inc | |
parent | 50106435e9a5bbc067e9beeabc5147ed9f305286 (diff) |
replace custom list with std::forward_list
cppcheck warns with publicAllocationError in
INetMIMECharsetList_Impl::prepend(...)
> { m_pFirst = new Node(rCharset, m_pFirst); }<--- Possible leak
> in public function. The pointer 'm_pFirst' is not deallocated before
> it is allocated.
We got a single linked list here which is correctly deallocated
in dtor.
But with C++11 std::forward_list there is no reason to implement or own
list.
Change-Id: Ia6582061c1e5f1284bd81fafef88ce475ee891aa
Reviewed-on: https://gerrit.libreoffice.org/31132
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'basic/source/inc')
0 files changed, 0 insertions, 0 deletions