diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2019-05-23 12:21:36 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2019-05-23 16:49:15 +0200 |
commit | 3c815c9785047761eff7e2895764c81442279976 (patch) | |
tree | 02bb2555a969a77aa441b5bc05e57309fd025c58 /framework/source/inc | |
parent | ab8b3172d20eb8d9e4a4de765bb97d7e795419d8 (diff) |
No need for AcceleratorCache::takeOver...
...as it does the same as the copy assignment op. And both of those are
apparently already only called with SolarMutex locked, so no need to lock it
again. So the copy assignment op (as well as the other special memeber
functions) can be left implicitly-declared. (And some uses of the original
takeOver can be optimized to use move assignment.)
Change-Id: I279a5e3ee85ff2342d6ef5f672108a0712fe116d
Reviewed-on: https://gerrit.libreoffice.org/72831
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'framework/source/inc')
-rw-r--r-- | framework/source/inc/accelerators/acceleratorcache.hxx | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/framework/source/inc/accelerators/acceleratorcache.hxx b/framework/source/inc/accelerators/acceleratorcache.hxx index fc295ae004dc..83ba071fd5db 100644 --- a/framework/source/inc/accelerators/acceleratorcache.hxx +++ b/framework/source/inc/accelerators/acceleratorcache.hxx @@ -76,26 +76,6 @@ class AcceleratorCache // interface public: - - /** @short creates a new - but empty - cache instance. */ - AcceleratorCache(); - - /** @short make a copy of this cache. - @descr Used for the copy-on-write feature. - */ - AcceleratorCache(const AcceleratorCache& rCopy); - - /** @short write changes back to the original container. - - @param rCopy - the (changed!) copy, which should be written - back to this original container. - */ - void takeOver(const AcceleratorCache& rCopy); - - /** TODO document me */ - AcceleratorCache& operator=(const AcceleratorCache& rCopy); - /** @short checks if the specified key exists. @param aKey |