Annoying warnings

View: New views
2 Messages — Rating Filter:   Alert me  

Annoying warnings

by Julien PUYDT :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi,

I fixed quite a few warnings, but there are still a few annoying ones :

A series of ambiguous 'else', where I'd rather see the code author add
braces, to be sure :
videooutput-manager-common.cpp:159
videooutput-manager-common.cpp:171
xwindow.cpp:749
xwindow.cpp:868

The compiler suggests parentheses around && within || in main.cpp:4152,
and again I'd prefer the code author to check where to put the
parentheses, just to be sure.

That one I have no idea how to fix... I thought forward declarations
where something normal :
endpoints/sip.h:56: warning: declaration ‘struct Ekiga::PersonalDetails’
does not declare anything

Snark
_______________________________________________
Ekiga-devel-list mailing list
Ekiga-devel-list@...
http://mail.gnome.org/mailman/listinfo/ekiga-devel-list

Re: Annoying warnings

by Matthias Schneider-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

They should be fixed now.
Matthias


--- Julien Puydt <jpuydt@...> schrieb am Fr, 6.6.2008:

> Von: Julien Puydt <jpuydt@...>
> Betreff: [Ekiga-devel-list] Annoying warnings
> An: "Ekiga development mailing list" <ekiga-devel-list@...>
> Datum: Freitag, 6. Juni 2008, 17:44
> Hi,
>
> I fixed quite a few warnings, but there are still a few
> annoying ones :
>
> A series of ambiguous 'else', where I'd rather
> see the code author add
> braces, to be sure :
> videooutput-manager-common.cpp:159
> videooutput-manager-common.cpp:171
> xwindow.cpp:749
> xwindow.cpp:868
>
> The compiler suggests parentheses around && within
> || in main.cpp:4152,
> and again I'd prefer the code author to check where to
> put the
> parentheses, just to be sure.
>
> That one I have no idea how to fix... I thought forward
> declarations
> where something normal :
> endpoints/sip.h:56: warning: declaration ‘struct
> Ekiga::PersonalDetails’
> does not declare anything
>
> Snark
> _______________________________________________
> Ekiga-devel-list mailing list
> Ekiga-devel-list@...
> http://mail.gnome.org/mailman/listinfo/ekiga-devel-list


      __________________________________________________________
Gesendet von Yahoo! Mail.
Dem pfiffigeren Posteingang.
http://de.overview.mail.yahoo.com
_______________________________________________
Ekiga-devel-list mailing list
Ekiga-devel-list@...
http://mail.gnome.org/mailman/listinfo/ekiga-devel-list
LightInTheBox - Buy quality products at wholesale price