Re: Questions about function lcl_GetMergeRange sc/source/core/data/fillinfo.cxx

2013-01-02 Thread Julien Nabet
On 29/12/2012 19:31, Markus Mohrhard wrote: Hey, In function lcl_GetMergeRange sc/source/core/data/fillinfo.cxx, what's the use to process "bHOver" variable in while (bVOver) ? It is unused. Just copy pasted code from the bHOver case. Ok, here's the change: https://gerrit.libreoffice.org/git

Re: Questions about function lcl_GetMergeRange sc/source/core/data/fillinfo.cxx

2012-12-29 Thread Markus Mohrhard
Hey, > In function lcl_GetMergeRange sc/source/core/data/fillinfo.cxx, what's the > use to process "bHOver" variable in while (bVOver) ? It is unused. Just copy pasted code from the bHOver case. > I thought about these too: > - declaring nOverlap before the 2 loops, it would avoid to declare it