qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v2] qxl: async I/O


From: Gerd Hoffmann
Subject: Re: [Qemu-devel] [PATCH v2] qxl: async I/O
Date: Fri, 08 Jul 2011 10:16:13 +0200
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.18) Gecko/20110617 Red Hat/3.1.11-2.el6_1 Thunderbird/3.1.11

btw, I'm looking at "#if.*MINOR" code like

  #if SPICE_INTERFACE_CORE_MINOR>= 3

(ui/spice-core.c)

Shouldn't that be checking the MAJOR as well?

major changing means a incompatible change. I doubt we ever will do that. But if you feel better checking that it probably should just be a

#if SPICE_INTERFACE_CORE_MAJOR != 1
#error incompatible spice core interface
#endif

at the top of the file.

cheers,
  Gerd




reply via email to

[Prev in Thread] Current Thread [Next in Thread]