]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
libvirt-domain: Drop virDomainMigrateCheckNotLocal attribute
authorMichal Privoznik <mprivozn@redhat.com>
Mon, 12 Oct 2015 14:37:15 +0000 (16:37 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Mon, 12 Oct 2015 14:37:15 +0000 (16:37 +0200)
Our apibuild.py script does not cope with ATTRIBUTE_NONNULL:

Parse Error: parsing function type, ')' expected
Got token  ('name', 'char')
Last token:  ('name', 'char')
Token queue:  [('op', '*'), ('name', 'dconnuri'), ('sep', ')')]
Line 3297 end:
Makefile:2441: recipe for target '../../docs/apibuild.py.stamp' failed

Let's drop it. Moreover, up until e17ae3ccc2dbc1400 where it was
introduced we did not really care about NULL-ity of dconnuri. And
moreover the ATTRIBUTE_NONNULL merely checks for static calls
over NULL, it won't catch the dynamic ones, where a NULL is
passed by a variable at runtime.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/libvirt-domain.c

index e6c8604e47c690858f6f7854c70b035c0d2eac96..6e1aacdd9589b31ddbf1f01e51de8ab2d6b29f32 100644 (file)
@@ -3293,7 +3293,7 @@ virDomainMigrateVersion3Params(virDomainPtr domain,
 }
 
 
-static int ATTRIBUTE_NONNULL(1)
+static int
 virDomainMigrateCheckNotLocal(const char *dconnuri)
 {
     virURIPtr tempuri = NULL;