summaryrefslogtreecommitdiff
path: root/svtools
diff options
context:
space:
mode:
authorTor Lillqvist <tml@collabora.com>2018-02-12 16:13:10 +0200
committerTor Lillqvist <tml@collabora.com>2018-05-16 21:01:21 +0200
commita48f22db7dd1bfeff62007a655566ddaad79aebd (patch)
treeb71702d8b028e06879952236f7453ab616bfb927 /svtools
parent851064d1b293e16372b5c1917c64f83eb5659e7c (diff)
Drop _Impl class name suffix where pointless
I.e. where there is no other LibreOffice, or system, or 3rd-party library thing with just the un-suffixed name that they would be a pimpl of, etc. Why bother calling something InterfaceOleWrapper_Impl when there is nothing that would be called InterfaceOleWrapper? It just makes the code seem more complicated and harder to read. An "implementation" of something is what everything that is not fully abstract *is*. We don't suffix the name of every non-abstract class with _Impl elsewhere either. Also drop two unused structs. Change-Id: I2755e6da78d8e4ded5291688dc90510b5f8b498b Reviewed-on: https://gerrit.libreoffice.org/54411 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
Diffstat (limited to 'svtools')
0 files changed, 0 insertions, 0 deletions