Hi Kohei,<br><br>Thanks that you simplified the code.<br><br>I moved the item from the "Easy Hacks" page to the "Completed" page.<br><br>I think now I will try "Make macro editing easier" or do you have (little) task?<br>
<br>Joost<br><br><div class="gmail_quote">2010/12/21 Kohei Yoshida <span dir="ltr"><<a href="mailto:kyoshida@novell.com">kyoshida@novell.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi Joost,<br>
<div class="im"><br>
On Mon, 2010-12-20 at 23:19 +0100, Joost Eekhoorn wrote:<br>
> Hi,<br>
><br>
> A patch for EasyHack "Leftover data after Undo operation" (qa issue<br>
> #106711).<br>
><br>
> Before pushing, please check if this patch:<br>
> - Does fix the problem.<br>
> - Do not introduce new issues. (Not fixing a problem and generated new<br>
> ones.)<br>
<br>
</div>Thanks for the patch. I believe that's the correct place to put a fix<br>
for this bug. And that change shouldn't affect other functionalities as<br>
far as I can tell.<br>
<br>
I committed and pushed your patch to master. I also made some follow-up<br>
changes to simplify the code a bit without changing the logic of the<br>
original code.<br>
<br>
Cheers,<br>
<br>
Kohei<br>
<font color="#888888"><br>
--<br>
Kohei Yoshida, LibreOffice hacker, Calc<br>
<<a href="mailto:kyoshida@novell.com">kyoshida@novell.com</a>><br>
<br>
</font></blockquote></div><br>