From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34421) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZrjUX-0002y2-Gu for qemu-devel@nongnu.org; Thu, 29 Oct 2015 05:27:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZrjUU-0005Zz-BL for qemu-devel@nongnu.org; Thu, 29 Oct 2015 05:27:29 -0400 Message-ID: <1446110842.5722.19.camel@redhat.com> From: Gerd Hoffmann Date: Thu, 29 Oct 2015 10:27:22 +0100 In-Reply-To: <5631D48D.90007@msgid.tls.msk.ru> References: <1438875392-9620-1-git-send-email-grangeleo@gmail.com> <56010629.3080307@msgid.tls.msk.ru> <5631D48D.90007@msgid.tls.msk.ru> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH][TRIVIAL] spice: fix checking for surface bpp match List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Tokarev Cc: qemu-trivial@nongnu.org, =?ISO-8859-1?Q?L=E9o?= Grange , qemu-devel On Do, 2015-10-29 at 11:10 +0300, Michael Tokarev wrote: > 22.09.2015 10:41, Michael Tokarev wrote: > > 17.09.2015 19:18, L=C3=A9o Grange wrote: > >> Ping, as I had no feedback about this small patch. > >> Please tell me if I need to do anything else. > >=20 > > Hello! > >=20 > > I've seen initial version of your patch, thought it isn't exactly tri= vial ;), > > and hoped someone will comment. Only now I realized you sent it to -= trivial > > but not to -devel, so there's no one to comment on it. Please don't = do this, > > always include qemu-devel when sending patches, so a patch will have = a > > chance to be reviewed. I've added qemu-devel to Cc now. > >=20 > > Here, I _think_ the patch is okay, but it'd be better for it to have = some > > review still. >=20 > Gerd, Cc'ing you this time, can you please take a look at this patch? Obsolete, commit b2af43cc379e1d4c30d92af257bedebf0e3f618a fixes this (and a few more cases). cheers, Gerd