diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-05 14:29:11 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-10 12:31:43 +0200 |
commit | 0fb5ca6cc9cc55a4436a36c533461769b1fc8526 (patch) | |
tree | 30e21bfcb0a6ad46f52a4e664728753c03d7a837 /include/tools | |
parent | 1a33947a91685808fd5f6d4903b6ae896686066d (diff) |
tdf#79877 perf loading docx file, writerfilter/ improvements
this improves load time by 20%.
We switch from shared_ptr to tools::SvRef to manage the objects
I noticed some double inheritance like this:
DomainMapper
LoggedProperties
Properties
SvRefBase
LoggedTable
Table
SvRefBase
so to be safe I made all the ref-count-base-class inheritance
virtual.
Change-Id: Ia3de9733f5c6966e8171f43d083dcc087040b8cd
Reviewed-on: https://gerrit.libreoffice.org/57022
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'include/tools')
-rw-r--r-- | include/tools/ref.hxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/include/tools/ref.hxx b/include/tools/ref.hxx index 68a1c3f56f09..62ac4952c6dc 100644 --- a/include/tools/ref.hxx +++ b/include/tools/ref.hxx @@ -130,11 +130,9 @@ protected: public: SvRefBase() : nRefCount(0), bNoDelete(1) {} - SvRefBase(const SvRefBase &) : nRefCount(0), bNoDelete(1) {} - SvRefBase & operator = ( const SvRefBase & ) - { return *this; } + SvRefBase & operator=(const SvRefBase &) { return *this; } void RestoreNoDelete() { bNoDelete = 1; } |