Re: [PATCH v5 0/2] Option to take screenshot with screendump as PNG

[Kshitij Suri]

Hi,

Hope this mail finds everyone well! I have updated the code as required 
and would be grateful if I could get your reviews for any changes that 
are needed to be implemented in the patch. In case no change is 
required, please do let me know the next steps for the same.

Regards,

Kshitij Suri

On 08/04/22 12:43 pm, Kshitij Suri wrote:
> This patch series aims to add PNG support using libpng to screendump method.
> Currently screendump only supports PPM format, which is uncompressed.
>
> PATCH 1 phases out CONFIG_VNC_PNG parameter and replaces it with CONFIG_PNG
> which detects libpng support.
>
> PATCH 2 contains core logic for PNG creation from pixman using libpng. HMP
> command equivalent is also implemented in this patch.
>
> v4->v5
>   - Modified format as a flag based optional parameter in HMP.
>
> v3->v4
>   - Added condition to check for libpng only in PNG option is allowed
>
> v2->v3
>   - HMP implementation fixes for png.
>   - Used enum for image format.
>   - Fixed description and updated QEMU support version.
>
> v1->v2:
>   - Removed repeated alpha conversion operation.
>   - Modified logic to mirror png conversion in vnc-enc-tight.c file.
>   - Added a new CONFIG_PNG parameter for libpng support.
>   - Changed input format to enum instead of string.
>   - Improved error handling.
>
> Kshitij Suri (2):
>    Replacing CONFIG_VNC_PNG with CONFIG_PNG
>    Added parameter to take screenshot with screendump as PNG
>
>   hmp-commands.hx    |  11 ++---
>   meson.build        |  12 +++---
>   meson_options.txt  |   4 +-
>   monitor/hmp-cmds.c |  12 +++++-
>   qapi/ui.json       |  24 +++++++++--
>   ui/console.c       | 101 +++++++++++++++++++++++++++++++++++++++++++--
>   ui/vnc-enc-tight.c |  18 ++++----
>   ui/vnc.c           |   4 +-
>   ui/vnc.h           |   2 +-
>   9 files changed, 157 insertions(+), 31 deletions(-)