On Mon, Aug 22, 2011 at 12:47:05PM -0700, Steven Dake wrote:
> This removes a sprintf operation in the totem and ipc logging operations
> 

ACK, will now need the latest libqb (else you can put a configure check
for that function).

-A

> Signed-off-by: Steven Dake <[email protected]>
> ---
>  exec/main.c |   13 +++----------
>  1 files changed, 3 insertions(+), 10 deletions(-)
> 
> diff --git a/exec/main.c b/exec/main.c
> index a9a3e3e..fde77da 100644
> --- a/exec/main.c
> +++ b/exec/main.c
> @@ -1055,17 +1055,10 @@ _logsys_log_printf(int level, int subsys,
>       size_t len;
>  
>       va_start(ap, format);
> -     len = vsnprintf(buf, sizeof(buf), format, ap);
> -     va_end(ap);
> -
> -     if (buf[len - 1] == '\n') {
> -             buf[len - 1] = '\0';
> -             len -= 1;
> -     }
> -
> -     qb_log_from_external_source(function_name, file_name,
> +     qb_log_from_external_source_va(function_name, file_name,
>                                   format, level, file_line,
> -                                 subsys, buf);
> +                                 subsys, ap);
> +     va_end(ap);
>  }
>  
>  static void fplay_key_change_notify_fn (
> -- 
> 1.7.6
> 
> _______________________________________________
> Openais mailing list
> [email protected]
> https://lists.linux-foundation.org/mailman/listinfo/openais
_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to