<div dir="ltr">Hi, we fixed the patch and send again.<div><br></div><div>Thanks.</div><div><br></div><div>Att Junior</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/10/1 Andrzej Hunt <span dir="ltr"><<a href="mailto:andrzej@ahunt.org" target="_blank">andrzej@ahunt.org</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Junior,<br>
<br>
Unfortunately your patch includes a lot of unnecessary reformatting<br>
(space indentation replaced with tab-stops),<br>
specifically in SlideShowActivity.java it's impossible to see what has<br>
changed since the whole file has had all space-indentation replaced with<br>
tab-indentation (i.e. git thinks the whole file has been removed and<br>
recreated).<br>
<br>
(There are also a few cases of lines with trailing space which the git<br>
commit-hooks would usually complain about.)<br>
<br>
No idea which editor/IDE you use, but if you could change back to space<br>
indentation that would be hugely simplify reviewing the patch -- it<br>
should probably be enough to configure it to use spaces for indentation<br>
(four spaces per tab) and then reformat the file (I'm guessing you might<br>
be using Eclipse?) which would remove most of the reformatting in the<br>
patch.<br>
<br>
Cheers,<br>
<br>
Andrzej<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, 2013-10-01 at 14:49 -0300, Junior Cesar Oliveira wrote:<br>
> I declare that all of my past & future contributions to LibreOffice<br>
> may be licensed under<br>
> the MPL/LGPLv3+ dual license.<br>
><br>
><br>
> Hello, the patch is attached to the resolution of bug 61570 SDremote<br>
> project. The patch has been created for the following academic<br>
> UTFPR-Brazil: Junior Cesar de Oliveira, Ana Claudia Maciel, Willyan<br>
> Schultz Dworak.<br>
><br>
><br>
><br>
><br>
><br>
> Junior.<br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<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>
</div></div></blockquote></div><br></div>