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

Reply via email to