|
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
[Prev in Thread] | Current Thread | [Next in Thread] |