Re: [PATCH] server: Remove unneeded asserts in mjpeg_encoder_get_stats()

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

 



On Mon, Nov 23, 2015 at 12:07:49PM -0500, Frediano Ziglio wrote:
> > 
> > No other function checks the 'this' pointer and if not given a buffer
> > to return the statistics we can simply not return any.
> > 
> > Signed-off-by: Francois Gouget <fgouget@xxxxxxxxxxxxxxx>
> > ---
> >  server/mjpeg_encoder.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > So following the logging discussion it looks like we would be going
> > towards using the g_return_xxx() functions so in
> > mjpeg_encoder_get_bit_rate()'s case we would get something like this.
> > Makes sense?
> > 
> 
> Ehm... no, NACK

Some rationale is needed so that Francois knows how to improve his
patch. Is the assert->return_if_fail change wrong and unwanted? Should
he use some other variant of return_if_fail?

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]