Re: [PATCH spice-server v3 0/6] Separate file statistic management

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



For the series:
Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>

Christophe

On Wed, Nov 16, 2016 at 03:39:10PM +0000, Frediano Ziglio wrote:
> Changes since v2:
> - multiple comments here and there;
> - add a patch to use g_strdup_printf instead of malloc/sprintf;
> - fix some typos;
> - compatibility with RHEL6 (test);
> - make RedStatFile private;
> - fix some leaks (one in the test).
> Thanks to Christophe.
> 
> Frediano Ziglio (6):
>   Separate code to manage statistic file
>   Fix node removal
>   Add a base test for statistic file
>   Use g_strdup_printf instead of manually malloc/snprintf
>   Avoid leaking file descriptor for statistics
>   Make RedStatFile private inside stat-file.c
> 
>  server/Makefile.am            |   2 +
>  server/reds-private.h         |  12 +--
>  server/reds.c                 | 122 ++---------------------
>  server/stat-file.c            | 223 ++++++++++++++++++++++++++++++++++++++++++
>  server/stat-file.h            |  39 ++++++++
>  server/stat.h                 |   4 +-
>  server/tests/Makefile.am      |   1 +
>  server/tests/stat-file-test.c | 105 ++++++++++++++++++++
>  8 files changed, 381 insertions(+), 127 deletions(-)
>  create mode 100644 server/stat-file.c
>  create mode 100644 server/stat-file.h
>  create mode 100644 server/tests/stat-file-test.c
> 
> -- 
> 2.7.4
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]