<div dir="auto"><pre style="white-space:pre-wrap;font-family:monospace,monospace;background-color:rgb(248,249,250);border:1px solid rgb(234,236,240);padding:1em;line-height:1.3em;font-size:14px">All of my past & future contributions to LibreOffice may be
licensed under the MPLv2/LGPLv3+ dual license.</pre></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 23 Aug, 2020, 11:57 PM Julien Nabet (via Code Review), <<a href="mailto:gerrit@gerrit.libreoffice.org">gerrit@gerrit.libreoffice.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Julien Nabet has posted comments on this change. ( <a href="https://gerrit.libreoffice.org/c/core/+/101242" rel="noreferrer noreferrer" target="_blank">https://gerrit.libreoffice.org/c/core/+/101242</a> )<br>
<br>
Change subject: tdf#87963: Add percentage value to new calculated compressed image size<br>
......................................................................<br>
<br>
<br>
Patch Set 2: Code-Review-2<br>
<br>
I'm not an expert but the main problem I see is you use "printf". The goal of the bugtracker is to display the information on UI not on a console.<br>
<br>
<br>
-- <br>
To view, visit <a href="https://gerrit.libreoffice.org/c/core/+/101242" rel="noreferrer noreferrer" target="_blank">https://gerrit.libreoffice.org/c/core/+/101242</a><br>
To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.libreoffice.org/settings" rel="noreferrer noreferrer" target="_blank">https://gerrit.libreoffice.org/settings</a><br>
<br>
Gerrit-Project: core<br>
Gerrit-Branch: master<br>
Gerrit-Change-Id: I5fb87fd4d8cceb38775cb98520c0c7ba1d49bc93<br>
Gerrit-Change-Number: 101242<br>
Gerrit-PatchSet: 2<br>
Gerrit-Owner: Anshu <<a href="mailto:anshukhare50@gmail.com" target="_blank" rel="noreferrer">anshukhare50@gmail.com</a>><br>
Gerrit-Reviewer: Jenkins<br>
Gerrit-Reviewer: Julien Nabet <<a href="mailto:serval2412@yahoo.fr" target="_blank" rel="noreferrer">serval2412@yahoo.fr</a>><br>
Gerrit-Comment-Date: Sun, 23 Aug 2020 18:27:20 +0000<br>
Gerrit-HasComments: No<br>
Gerrit-Has-Labels: Yes<br>
Gerrit-MessageType: comment<br>
</blockquote></div>