[SSSD] [PATCH] KRB5: Fix warning declaration shadows global declaration

Jakub Hrozek jhrozek at redhat.com
Tue Sep 3 18:54:39 UTC 2013


On Tue, Sep 03, 2013 at 01:47:59PM +0200, Lukas Slebodnik wrote:
> On (03/09/13 13:34), Jakub Hrozek wrote:
> >On Tue, Sep 03, 2013 at 01:09:55PM +0200, Sumit Bose wrote:
> >> On Mon, Sep 02, 2013 at 06:25:50PM +0200, Lukas Slebodnik wrote:
> >> > On (02/09/13 17:50), Sumit Bose wrote:
> >> > >On Mon, Sep 02, 2013 at 05:43:06PM +0200, Lukas Slebodnik wrote:
> >> > >> ehlo,
> >> > >> 
> >> > >> I tested some patches on RHEL6. and there is newly introduced warning.
> >> > >> gcc version 4.4.7
> >> > >> 
> >> > >> src/providers/krb5/krb5_utils.c:193: warning: declaration of 'rewind' shadows a
> >> > >> global declaration
> >> > >> /usr/include/stdio.h:754: warning: shadowed declaration is here
> >> > >> 
> >> > >> Warning was introduced in patch dcc6877aa2e2dd63a9dc9c411a9c58feaeb36b9a
> >> > >> "krb5: Fetch ccname template from krb5.conf"
> >> > >> 
> >> > >> LS
> >> > >
> >> > >ACK. I was about to send the same patch, just with a different name :-)
> >> > >
> >> > >bye,
> >> > >Sumit
> >> > I renamed "rewind" -> "want_rewind"
> >> > What was your proposal? Maybe your will be better.
> >> 
> >> I used 'rerun' the name of the little brother of Linus (van Pelt not
> >> Torvalds :-)
> >> 
> >> bye,
> >> Sumit
> >
> >I like the shorter name better.
> 
> This was a reason why I asked.
> 
> Attached patch contains "shorter version"
> 
> LS

ACK



More information about the sssd-devel mailing list