diff options
author | Julien Nabet <serval2412@yahoo.fr> | 2019-11-26 18:40:04 +0100 |
---|---|---|
committer | Julien Nabet <serval2412@yahoo.fr> | 2019-11-26 23:31:56 +0100 |
commit | d8feeecfec0753b566183d30d935bd389a9c9515 (patch) | |
tree | 9938c693474473d3e94a132f13143d212d07072d /connectivity | |
parent | ffa636ba74b04b3258ec9a696bc4eac33581fa24 (diff) |
Fix for 2 commits related to cppcheck:
1) https://cgit.freedesktop.org/libreoffice/core/commit/?id=4e1ae2a9a0d2d9185b49677fa4ea2a2b1fe8bab2
I had let "size" and "fields" initialized twice instead of initializing only
2) https://cgit.freedesktop.org/libreoffice/core/commit/?id=e6ab01ce532d1db01579b70bd476b2f643522bf9
put back original order for NestedKeyImpl and adjust initialization accordingly
Change-Id: I6ca4e6520558011c359f8ea089b6d431ad79edfe
Reviewed-on: https://gerrit.libreoffice.org/83810
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Diffstat (limited to 'connectivity')
-rw-r--r-- | connectivity/source/drivers/macab/MacabRecord.cxx | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/connectivity/source/drivers/macab/MacabRecord.cxx b/connectivity/source/drivers/macab/MacabRecord.cxx index b7e270f378f2..3072e1eaa2f4 100644 --- a/connectivity/source/drivers/macab/MacabRecord.cxx +++ b/connectivity/source/drivers/macab/MacabRecord.cxx @@ -42,8 +42,6 @@ MacabRecord::MacabRecord(const sal_Int32 _size) : size(_size), fields(std::make_unique<macabfield *[]>(size)) { - size = _size; - fields = std::make_unique<macabfield *[]>(size); sal_Int32 i; for(i = 0; i < size; i++) fields[i] = nullptr; |