Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


Hi Junior,

Unfortunately your patch includes a lot of unnecessary reformatting
(space indentation replaced with tab-stops),
specifically in SlideShowActivity.java it's impossible to see what has
changed since the whole file has had all space-indentation replaced with
tab-indentation (i.e. git thinks the whole file has been removed and
recreated).

(There are also a few cases of lines with trailing space which the git
commit-hooks would usually complain about.)

No idea which editor/IDE you use, but if you could change back to space
indentation that would be hugely simplify reviewing the patch -- it
should probably be enough to configure it to use spaces for indentation
(four spaces per tab) and then reformat the file (I'm guessing you might
be using Eclipse?) which would remove most of the reformatting in the
patch.

Cheers,

Andrzej

On Tue, 2013-10-01 at 14:49 -0300, Junior Cesar Oliveira wrote:
I declare that all of my past & future contributions to LibreOffice
may be licensed under
the MPL/LGPLv3+ dual license.


Hello, the patch is attached to the resolution of bug 61570 SDremote
project. The patch has been created for the following academic
UTFPR-Brazil: Junior Cesar de Oliveira, Ana Claudia Maciel, Willyan
Schultz Dworak.





Junior.
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice



Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.