[PATCH] [core/vcl/source/window/splitwin.cxx:2047] -> [core/vcl/source/window/splitwin.cxx:2045]: (style) Found duplicate branches for if and else.
mariuszdykierek at gmail.com
Thu Feb 16 02:46:37 PST 2012
On 2012-02-16 10:02, Stephan Bergmann wrote:
> On 02/16/2012 09:35 AM, Riccardo Magliocchetti wrote:
>> Otherwise you can simplify it even more:
>> sal_Bool bLeft = (meAlign == WINDOWALIGN_TOP || meAlign ==
>> WINDOWALIGN_LEFT) ? sal_False : sal_True;
> ... which of course reduces to
>> bool bLeft = !(meAlign == WINDOWALIGN_TOP || meAlign == WINDOWALIGN_LEFT);
>> bool bLeft = meAlign == WINDOWALIGN_RIGHT || meAlign == WINDOWALIGN_BOTTOM
> given that WindowAlign has exactly those four members (and it makes the name "bLeft" look
I personally find 'if' more legible than ?: and definitely expressions like b = x==y || x==z;
I am not sure if WindowAlign will always have only these 4 members and possibly the author of
the original version wasn't either (thus final else).
Since I am a newbie here, I would vote for a simple 'if' or a 'switch'.
Let me know what is the decision and I will change the code accordingly or... feel free to
change the code and provide an alternative patch.
More information about the LibreOffice