On Fri, Feb 14, 2014 at 06:32:21AM -0700, Eric Blake wrote: > On 02/14/2014 06:23 AM, Martin Kletzander wrote: > > On Thu, Feb 13, 2014 at 07:51:43PM +0100, Michal Privoznik wrote: > >> This is also adding new ACL permission to check 'set_time'. > >> > >> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> > >> --- > > > > > Applies to previous patch too, this 'time' will be a problem with > > '-Wshadow-declarations' on older (some) compilers. > > > > ACK with that variable name changed. > > I'm half-tempted to just tweak m4/virt-compile-warnings.m4 to drop > -Wshadow-declarations on older gcc. Since newer gcc is sane about local > variables not conflicting with public functions, it's not worth worrying > about the collisions that only older gcc reports. The problem is shadow decls can occur within libvirt code too in which case they would likely be genuine bugs. eg someone declares 'foo' at the start of a method and some time later redeclares it in a for/while loop body or some such. Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list