Re: [PATCH spice-common] marshaller: Remove initial underscore from static function

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

 



On 18/05/18 16:09, Frediano Ziglio wrote:
> This is the only function starting with an underscore, looks
> out of style.
> 
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

Acked-by: Eduardo Lima (Etrunko) <etrunko@xxxxxxxxxx>

> ---
>  common/marshaller.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/common/marshaller.c b/common/marshaller.c
> index 80cedb7..b55b568 100644
> --- a/common/marshaller.c
> +++ b/common/marshaller.c
> @@ -270,7 +270,7 @@ static size_t remaining_buffer_size(SpiceMarshallerData *d)
>      return MARSHALLER_BUFFER_SIZE - d->current_buffer_position;
>  }
>  
> -static void _reserve_space_free_data(uint8_t *data, SPICE_GNUC_UNUSED void *opaque)
> +static void reserve_space_free_data(uint8_t *data, SPICE_GNUC_UNUSED void *opaque)
>  {
>      free(data);
>  }
> @@ -313,7 +313,7 @@ uint8_t *spice_marshaller_reserve_space(SpiceMarshaller *m, size_t size)
>          /* Large item, allocate by itself */
>          item->data = (uint8_t *)spice_malloc(size);
>          item->len = size;
> -        item->free_data = _reserve_space_free_data;
> +        item->free_data = reserve_space_free_data;
>          item->opaque = NULL;
>      } else {
>          /* Use next buffer */
> 


-- 
Eduardo de Barros Lima (Etrunko)
Software Engineer - RedHat
etrunko@xxxxxxxxxx
_______________________________________________
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]