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.