gcc 4.4 warnings...

[Gilles Caulier-4]

Andi,

Do you see these warnings :

/home/gilles/Documents/data/devel/SVN/trunk/graphics/digikam/utilities/fuzzysearch/fuzzysearchview.cpp:
In member function ‘void
Digikam::FuzzySearchView::newDuplicatesSearch(Digikam::Album*)’:
/home/gilles/Documents/data/devel/SVN/trunk/graphics/digikam/utilities/fuzzysearch/fuzzysearchview.cpp:498:
attention : enumeration value ‘DATE’ not handled in switch
/home/gilles/Documents/data/devel/SVN/trunk/graphics/digikam/utilities/fuzzysearch/fuzzysearchview.cpp:498:
attention : enumeration value ‘SEARCH’ not handled in switch

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Yes,

didn't know you need to handle every possible switch case. I use an if-
statement now.

Gilles,

can you compile digiKam at the moment? I'm not able to link against Nepomuk
methods (using KDE 4.2.4).

Andi

On Wednesday 24 June 2009 08:07:19 Gilles Caulier wrote:

> Andi,
>
> Do you see these warnings :
>
> /home/gilles/Documents/data/devel/SVN/trunk/graphics/digikam/utilities/fuzz
>ysearch/fuzzysearchview.cpp: In member function ‘void
> Digikam::FuzzySearchView::newDuplicatesSearch(Digikam::Album*)’:
> /home/gilles/Documents/data/devel/SVN/trunk/graphics/digikam/utilities/fuzz
>ysearch/fuzzysearchview.cpp:498: attention : enumeration value ‘DATE’ not
> handled in switch
> /home/gilles/Documents/data/devel/SVN/trunk/graphics/digikam/utilities/fuzz
>ysearch/fuzzysearchview.cpp:498: attention : enumeration value ‘SEARCH’ not
> handled in switch
>
> Gilles
> _______________________________________________
> Digikam-devel mailing list
> [hidden email]
> https://mail.kde.org/mailman/listinfo/digikam-devel

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

2009/6/24 Andi Clemens <[hidden email]>:
> Yes,
>
> didn't know you need to handle every possible switch case. I use an if-
> statement now.
>
> Gilles,
>
> can you compile digiKam at the moment? I'm not able to link against Nepomuk
> methods (using KDE 4.2.4).

yes, all link fine here, using KDE 4.2.2 from Mandriva 2009.1 (official).

I do nothing special.

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

On Wednesday 24 June 2009 11:48:54 am Andi Clemens wrote:

> didn't know you need to handle every possible switch case. I use an if-
> statement now.
>

You only get the warning if you do not have a "default" in the switch (not
handling all cases). Adding a "default" case will remove the warning.

I think it is a great feature that it warns you if you have forgotten a switch
case :)

Kåre



_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Yes, but I don't like having
default: break;
in the code :-) An if is shorter here...

Is this a new feature of gcc4.4?
Never seen this before.

Andi

On Wednesday 24 June 2009 22:29:55 Kåre Särs wrote:

> On Wednesday 24 June 2009 11:48:54 am Andi Clemens wrote:
> > didn't know you need to handle every possible switch case. I use an if-
> > statement now.
>
> You only get the warning if you do not have a "default" in the switch (not
> handling all cases). Adding a "default" case will remove the warning.
>
> I think it is a great feature that it warns you if you have forgotten a
> switch case :)
>
> Kåre
>
>
>
> _______________________________________________
> Digikam-devel mailing list
> [hidden email]
> https://mail.kde.org/mailman/listinfo/digikam-devel

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

On Thursdayen den 25 June 2009 00:05:31 Andi Clemens wrote:
> Yes, but I don't like having
> default: break;
> in the code :-) An if is shorter here...
>
> Is this a new feature of gcc4.4?
> Never seen this before.
>

I think it has been there for awhile, I don't know when :)

Kåre
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

> Yes, but I don't like having
> default: break;
> in the code :-) An if is shorter here...
>
> Is this a new feature of gcc4.4?
> Never seen this before.
>

I have had it for a long time as well. It is sometimes slightly annoying, but
there are cases where it is useful.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel