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


From Philipp Riemer <ruderphilipp@gmail.com>:

Philipp Riemer has posted comments on this change.

Change subject: add/remove blank lines and braces (in sw/source/core/crsr/)
......................................................................


Patch Set 1:

To be honest, I don't understand how these removals differ from 
<https://gerrit.libreoffice.org/#/c/274/>. I mean by removing commented code I might also introduce 
merge conflicts, or not?

Or is your comment only because I added braces in some parts to make the code more readable? If so, 
would it help to provide a second version (at the moment I am not sure how) with only removed code 
but nothing added?

--
To view, visit https://gerrit.libreoffice.org/272
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4dea8ea84d83c28eb258d0e8ddc2966476a50223
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Philipp Riemer <ruderphilipp@gmail.com>
Gerrit-Reviewer: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Gerrit-Reviewer: Philipp Riemer <ruderphilipp@gmail.com>


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.