[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Linphone-users] ortp: fix some gcc warnings
From: |
Simon Morlat |
Subject: |
Re: [Linphone-users] ortp: fix some gcc warnings |
Date: |
Thu, 31 Aug 2006 13:26:32 +0200 |
User-agent: |
KMail/1.9.3 |
Of course I'll accept this patch.
I would prefer that you use the cvs tree and cvs diff to generate it, so that
I'll be sure of having no problems in applying it (was not the case of
previous patch, but as it was trivial I fixed it manually).
Simon
Le mercredi 30 août 2006 10:22, Andre Noll a écrit :
> On 21:09, Simon Morlat wrote:
> > Thanks, it is now applied to cvs.
>
> Fine. There are some more annoying warnings that show up when using
> gcc's -Wshadow option (which is generally considered a good idea(TM)).
>
> This is due to ortp using "index", "signal" and "time" as names for
> variables while these have also global declarations.
>
> Would you accept another patch that cleans this up? If so, would you
> prefer to have only changes in the header files, or should I change the
> .c files as well to match the names in the headers?
>
> Andre