All of my past & future contributions to LibreOffice may be licensed under the MPLv2/LGPLv3+ dual license.
On Sun, 23 Aug, 2020, 11:57 PM Julien Nabet (via Code Review), < ger...@gerrit.libreoffice.org> wrote: > Julien Nabet has posted comments on this change. ( > https://gerrit.libreoffice.org/c/core/+/101242 ) > > Change subject: tdf#87963: Add percentage value to new calculated > compressed image size > ...................................................................... > > > Patch Set 2: Code-Review-2 > > 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. > > > -- > To view, visit https://gerrit.libreoffice.org/c/core/+/101242 > To unsubscribe, or for help writing mail filters, visit > https://gerrit.libreoffice.org/settings > > Gerrit-Project: core > Gerrit-Branch: master > Gerrit-Change-Id: I5fb87fd4d8cceb38775cb98520c0c7ba1d49bc93 > Gerrit-Change-Number: 101242 > Gerrit-PatchSet: 2 > Gerrit-Owner: Anshu <anshukhar...@gmail.com> > Gerrit-Reviewer: Jenkins > Gerrit-Reviewer: Julien Nabet <serval2...@yahoo.fr> > Gerrit-Comment-Date: Sun, 23 Aug 2020 18:27:20 +0000 > Gerrit-HasComments: No > Gerrit-Has-Labels: Yes > Gerrit-MessageType: comment >
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice