diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-01-16 14:13:31 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-01-17 11:31:40 +0000 |
| commit | 70fed865df7655a7ee65fa6cde51bbf93182dbbb (patch) | |
| tree | 00c77d8ba47be246cd247697045fdcfe6df3ad9e /include | |
| parent | fb4b59f9370167696d1a67beb16b593ba86971a8 (diff) | |
new loplugin: useuniqueptr: forms..framework
Change-Id: I4300a13f455148b7156ac3f444c7102d63ae6db3
Reviewed-on: https://gerrit.libreoffice.org/33164
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'include')
| -rw-r--r-- | include/formula/token.hxx | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/include/formula/token.hxx b/include/formula/token.hxx index a2376bc0fc28..2405b66b2e4e 100644 --- a/include/formula/token.hxx +++ b/include/formula/token.hxx @@ -388,22 +388,23 @@ public: class FORMULA_DLLPUBLIC FormulaJumpToken : public FormulaToken { private: - short* pJump; + std::unique_ptr<short[]> + pJump; bool bIsInForceArray; public: FormulaJumpToken( OpCode e, short* p ) : FormulaToken( formula::svJump , e), bIsInForceArray( false) { - pJump = new short[ p[0] + 1 ]; - memcpy( pJump, p, (p[0] + 1) * sizeof(short) ); + pJump.reset( new short[ p[0] + 1 ] ); + memcpy( pJump.get(), p, (p[0] + 1) * sizeof(short) ); } FormulaJumpToken( const FormulaJumpToken& r ) : FormulaToken( r ), bIsInForceArray( r.bIsInForceArray) { - pJump = new short[ r.pJump[0] + 1 ]; - memcpy( pJump, r.pJump, (r.pJump[0] + 1) * sizeof(short) ); + pJump.reset( new short[ r.pJump[0] + 1 ] ); + memcpy( pJump.get(), r.pJump.get(), (r.pJump[0] + 1) * sizeof(short) ); } virtual ~FormulaJumpToken() override; virtual short* GetJump() const override; |