Hi Thomas,<br><br>Thanks for the correction.<br><br>Balint<br><br><div class="gmail_quote">2011/2/14 Thomas Arnhold <span dir="ltr"><<a href="mailto:thomas@arnhold.org">thomas@arnhold.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Balint,<br>
<br>
looks good. I've pushed the second one. I slightly changed your patch -<br>
removed the newlines for the empty methods and didn't remove the<br>
description in writerfilter/inc/doctok/WW8Document.hxx as this is some<br>
kind of doxygen code.<br>
<br>
For the first patch, which removes the method, I'm not fully sure. I've<br>
applied it locally and it compiles fine. But it would be great if<br>
someone else could review this, too.<br>
<br>
Thomas<br>
<div><div></div><div class="h5"><br>
On 02/14/2011 12:49 PM, Bálint Dózsa wrote:<br>
> Hy,<br>
><br>
> I have removed some dead code (0002-Remove-dead-code.patch), and one<br>
> unused method (0001-Remove-unused-method-GetListTable.patch).<br>
><br>
> Regards,<br>
> Balint Dozsa<br>
><br>
><br>
><br>
</div></div>> _______________________________________________<br>
> LibreOffice mailing list<br>
> <a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
<br>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org">LibreOffice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
</blockquote></div><br>