[Spice-devel] [PATCH spice 1/12] server: Convert a couple of rate control checks into asserts in the mjpeg video encoder. (take 3)

Christophe Fergeau cfergeau at redhat.com
Wed Jun 24 00:36:05 PDT 2015


Hey,

On Mon, Jun 22, 2015 at 06:17:43PM +0200, Francois Gouget wrote:
> On Wed, 10 Jun 2015, Francois Gouget wrote:
> 
> > The checks would lead the reader to think these functions can be called when bit rate control is off when in fact they are only called when it is active.
> > 
> > Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
> > ---
> > 
> > This patch stands on its own and I think it makes sense even if the 
> > remainder of the series is not applied. It seemed ok in round 1 so 
> > again no change this time around. Let me know if it needs further 
> > refinements for inclusion.
> 
> This patch was acked a while ago but has not been committed yet.
> Does it need changes? Should I drop it from the series?
> (Same thing for the next one, 2/12)
> 
> http://lists.freedesktop.org/archives/spice-devel/2015-May/019843.html

Sorry for the delay, I've pushed it now.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20150624/786da4fa/attachment.sig>


More information about the Spice-devel mailing list