[Libreoffice] semantics of ScBaseCell* lclCloneCell

[Stephan Bergmann]

On 10/27/2011 07:53 PM, Kevin Hunter wrote:
> At 2:49am -0400 Thu, 27 Oct 2011, Stephan Bergmann wrote:
>> Not an sc expert, but from looking at the code it looks to me like the
>> default case shall never be reached, but is there to (a) silence
>> compiler warnings and (b) fire an OSL_FAIL in case there is a
>> programming error that causes the case to be reached nonetheless (I
>> usually put a "this cannot happen" into such OSL_FAILs, in good old
>> Knuth tradition). And since the function still needs to syntactically
>> return something, even in this "unreachable" case, it returns the
>> canonic null value.
>
> Huh. Then why use a switch? Why not use an if-else set of clauses? I
> recognize that this function may be called a lot, but does an if vs
> switch statement mean /that/ much in terms of performance?

But (a) a switch looks that much more cleaner than an if-else-cascade, 
and (b) you have the same situation with the if-else-cascade anyway, the 
default switch case corresponding to a final else branch there.

Stephan