Change in core[master]: add/remove blank lines and braces (in sw/source/core/crsr/)

Gerrit gerrit at gerrit.libreoffice.org
Mon Jul 9 05:21:05 PDT 2012


>From Philipp Riemer <ruderphilipp at 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 at gmail.com>
Gerrit-Reviewer: Björn Michaelsen <bjoern.michaelsen at canonical.com>
Gerrit-Reviewer: Philipp Riemer <ruderphilipp at gmail.com>



More information about the LibreOffice mailing list