removing unused functions

Norbert Thiebaud nthiebaud at gmail.com
Thu Nov 15 08:17:01 PST 2012


On Thu, Nov 15, 2012 at 5:24 AM, Enrico Weigelt <enrico.weigelt at vnc.biz> wrote:
> Hi folks,
>
> I'm currently working on removing unused functions (mentioned
> in unused.easy). Should I do it one by one with separate commits
> or in one big commit ?
>
> An big commit would make it easier/quicker for me (as I dont
> need so many rebuilds), but perhaps could make review a bit
> harder.

and increase the odd that the patch would get rejected for removing
something that _is_ really used (albeit not on linux)
the unused.easy thing is parsing a linux build... so there may be some
false positive there...

SO I would suggest smaller patch (rebuilding should not be that
painfull, if you stay on the same position on master and just do
incremental build...

Norbert


More information about the LibreOffice mailing list