Hi Cédric,<div><br></div><div>I&#39;m doing it task by task. This patches removes all bug id&#39;s from Oracle internal database keeping the indentation, you could apply it, even with the lines which the content is &quot;//&quot; &quot;- loop prevention&quot; or &quot;- optimization&quot;, i&#39;ll fix it in the next patch. ;-)</div>
<div><br></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8">Under the LGPLv3+ / MPL license.</div><div><br></div><div>Regards,</div><div>Júlio.<br><br><div class="gmail_quote">2011/4/15 Júlio Hoffimann <span dir="ltr">&lt;<a href="mailto:julio.hoffimann@gmail.com">julio.hoffimann@gmail.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Ok Cedric, no problem. I will fix these problems.<div><br></div><div>Thank you for return.</div><div><br></div><div>Regards,</div>
<div>Júlio.<div><div></div><div class="h5"><br><br><div class="gmail_quote">2011/4/15 Cedric Bosdonnat <span dir="ltr">&lt;<a href="mailto:cedric.bosdonnat.ooo@free.fr" target="_blank">cedric.bosdonnat.ooo@free.fr</a>&gt;</span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Julio,<br>
<br>
Many thanks for your patches nice patches. I think we&#39;ll manage to<br>
eradicate loads of useless comments this way ;)<br>
<br>
I pushed both patches in libs-gui and libs-core after some manual edits,<br>
but it takes too much time to manually adjust the patch for sw. Could<br>
you please adjust your regexps to fix the following problems:<br>
  * comments indentation is reduced to 4 spaces in all cases: you should<br>
keep the number of existing leading spaces to avoid breaking the<br>
indentation<br>
  * #b[0-9]\+# comments should be purely removed: we don&#39;t need them as<br>
they refer to some Oracle internal bug database. Please pay attention<br>
here ase you may have #b...# and #i....# references on the same line<br>
  * you often have some comment after the bug number, please keep them.<br>
Some could be removed, but not all.<br>
  * While at it, could you remove the lines with comments like &#39;// &lt;--&#39;<br>
as those are the &quot;closing&quot; comments for the other silly SCM-like<br>
comments?<br>
<br>
Don&#39;t forget to check your patch after running your regexps as you may<br>
have other unexpected cases.<br>
<br>
Regards,<br>
<div><div></div><div><br>
On Thu, 2011-04-14 at 19:26 -0300, Júlio Hoffimann wrote:<br>
&gt; I forget to mention...<br>
&gt;<br>
&gt;<br>
&gt; This patch is under the LGPLv3+ / MPL license.<br>
&gt;<br>
&gt; 2011/4/14 Júlio Hoffimann &lt;<a href="mailto:julio.hoffimann@gmail.com" target="_blank">julio.hoffimann@gmail.com</a>&gt;<br>
&gt;         ... and the last and big one. ;-)<br>
&gt;<br>
&gt;<br>
&gt;         <a href="https://bugs.freedesktop.org/show_bug.cgi?id=36234" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=36234</a><br>
&gt;<br>
&gt;<br>
&gt;         Regards,<br>
&gt;         Júlio.<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
</div></div>&gt; _______________________________________________<br>
&gt; LibreOffice mailing list<br>
&gt; <a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
&gt; <a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
<font color="#888888"><br>
--<br>
Cédric Bosdonnat<br>
LibreOffice hacker<br>
<a href="http://documentfoundation.org" target="_blank">http://documentfoundation.org</a><br>
OOo Eclipse Integration developer<br>
<a href="http://cedric.bosdonnat.free.fr" target="_blank">http://cedric.bosdonnat.free.fr</a><br>
<br>
</font></blockquote></div><br></div></div></div>
</blockquote></div><br></div>