Hi Adrien,
You can amend your commit locally, and resubmit. The patch will be
updated on Gerrit. Just make sure you don't change the Change-Id in the
commit message. That's how Gerrit identifies patches.
Regarding the alternative operators; they are 'alternatives' existing to
be be used when necessary.[0] And they require extra #include to be
compatible with C. So I would just stick with the usual representations
of the logical operators.
[0] https://en.cppreference.com/w/cpp/language/operator_alternative
Best,
Muhammet
--
Consultant Software Engineer
Collabora
On 5/3/19 1:25 AM, Regina Henschel wrote:
Hi Adrien,
Adrien Ollier schrieb am 02-May-19 um 21:32:
Hi everybody,
All of my commits fail at Jenkins tests:
[..}
https://gerrit.libreoffice.org/#/c/71652/
Wrong syntax for the boolean operator.
http://www.cplusplus.com/doc/tutorial/operators/
But the changes are very simple and on my computer I have no error.
Do you have option --enable-werror in your configure?
Kind regards
Regina
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://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.