[libvirt] [PATCH v1 2/4] remote: Implement remote{Get,Set}Time

Daniel P. Berrange berrange at redhat.com
Fri Feb 14 13:34:13 UTC 2014


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 at redhat.com>
> >> ---
> 
> > 
> > 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 :|




More information about the libvir-list mailing list