<div dir="ltr">Hi!<div><br></div><div>Thanks for taking a look, you are right! I think my mistake was that I forgot to negate the bools, which lead me to believe that bSkipX/Y were always true.</div><div><br></div><div>All the best,</div><div>Krisztian</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 22 June 2015 at 13:04, Eike Rathke <span dir="ltr"><<a href="mailto:erack@redhat.com" target="_blank">erack@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Krisztian,<br>
<span class=""><br>
On Sunday, 2015-06-21 22:11:44 +0200, Krisztian Pinter wrote:<br>
<br>
> I also think I found some dead code, if someone could shed some light on<br>
> the history of it, I would appreciate it. This is my patch which removes<br>
> it: <a href="https://gerrit.libreoffice.org/#/c/16351/" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/#/c/16351/</a><br>
> ScOutputData::DrawExtraShadow gets called with all false parameters which<br>
> causes it to just skip any sort of drawing. It seems to be leftovers from<br>
> some refactoring.<br>
<br>
</span>I don't think so. I haven't investigated thoroughly, but to me it looks<br>
like only the shadow of the surrounding outer cells is suppressed in the<br>
false case. All inner cells still have their shadow(s) drawn.<br>
<br>
    bool bSkipY = ( nArrY==0 && !bTop ) || ( nArrY+1 == nArrCount && !bBottom );<br>
    bool bSkipX = ( nArrX==nX1 && !bLeft ) || ( nArrX==nX2+2 && !bRight );<br>
<br>
does not skip for 0 < nArrY < nArrCount-1 and nX1 < nArrX < nX2+2<br>
<span class="HOEnZb"><font color="#888888"><br>
  Eike<br>
<br>
--<br>
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.<br>
GPG key "ID" 0x65632D3A - 2265 D7F3 A7B0 95CC 3918  630B 6A6C D5B7 6563 2D3A<br>
Better use 64-bit 0x6A6CD5B765632D3A here is why: <a href="https://evil32.com/" rel="noreferrer" target="_blank">https://evil32.com/</a><br>
Care about Free Software, support the FSFE <a href="https://fsfe.org/support/?erack" rel="noreferrer" target="_blank">https://fsfe.org/support/?erack</a><br>
</font></span></blockquote></div><br></div>