On Fri, Sep 30, 2011 at 3:26 AM, Caolán McNamara <caol...@redhat.com> wrote: > On Fri, 2011-09-30 at 02:15 -0500, Norbert Thiebaud wrote: >> When removing 'unused' code using unusedcode.easy, if you have an >> accessor pair GetFoo()/SetFoo() and only one of these is unused, >> please do not delete it. > > Why :-)
because then I won't have to resurrect the code when trying to merge patches... :-) but more generally: either the accessor is not useful and both set and get can go, or it is and the fact that one side is not used is more a signe that someone is not using it when it should or not using it _yet_ Norbert _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
