[Libreoffice] [PATCHES] [Janitor] Writer accessibility.cxx Cleanup/Bugfix

Phil Bordelon phil at thenexusproject.org
Thu Sep 30 08:54:13 PDT 2010


Folks,

Here are two patches for Writer/StarMath's accessibility.cxx.  (I
picked a random module and went for the first C++ file I could find
within.) One patch simply removes a whole bunch of dead code from
conditionals; the other is actually a minor bugfix, wherein someone
commented out a null-pointer check where they almost certainly just
meant to remove a refcounter.

I was wondering just how broad this patches should be; I have one that
removes a lot more dead code from accessibility.cxx, but should it also
do the same to the .hxx?  Other files at the same time?

These patches are released under the GPLv3+, if that's relevant for
something so basic.

P


More information about the LibreOffice mailing list