patches to clean up cppunit warnings (fdo#51154)

Markus Mohrhard markus.mohrhard at
Thu Jul 26 12:11:03 PDT 2012


2012/7/26 Robinson Tryon <bishop.robinson at>:
> Here are a few patches that make cppunit compile warning-free when
> configured with "-Weffc++" (and continue to be warning-free with
> "-Wall -Wextra -Werror", too!)
> The .gitignore patches are pretty straightforward. The other commits
> clean up a few odds and ends, order constructor initialization lists
> properly, etc. Later commits are to fix warnings revealed by 'make
> check'.
> When gcc complained about classes with pointer data members not
> overriding the default copy constructors and assignment operators, I
> first added private methods to prevent the use of these methods (being
> as conservative as possible); for any class that actually required
> working implementations, I added simple methods that implement the
> default (shallow-copy, etc.) behavior. I assume that some of these
> classes may need more than just stubs for these methods, but I figured
> that I'd get some feedback :-)

Great work. I'll have a look at them. This is a great step into
updating the source code to modern design principles.

Thanks a lot for this work.


More information about the LibreOffice mailing list