Compile warnings

Hi,

since 3 months I get a lot of compile warnings especially with libjpeg:

/home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:  
warning: enumeration value ‘JCS_EXT_RGB’ not handled in switch [-Wswitch]
/home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:  
warning: enumeration value ‘JCS_EXT_RGBX’ not handled in switch [-Wswitch]
/home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:  
warning: enumeration value ‘JCS_EXT_BGR’ not handled in switch [-Wswitch]
/home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:  
warning: enumeration value ‘JCS_EXT_BGRX’ not handled in switch [-Wswitch]
/home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:  
warning: enumeration value ‘JCS_EXT_XBGR’ not handled in switch [-Wswitch]
/home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:  
warning: enumeration value ‘JCS_EXT_XRGB’ not handled in switch [-Wswitch]

Can we handle those new variables with
default:break;

or do we need to fix our code more precisely?

My components:

digiKam version 2.0.0-beta5
Exiv2 can write to Jp2: Yes
Exiv2 can write to Jpeg: Yes
Exiv2 can write to Pgf: Yes
Exiv2 can write to Png: Yes
Exiv2 can write to Tiff: Yes
Exiv2 supports XMP metadata: Yes
LibCImg: 130
LibClapack: internal library
LibExiv2: 0.21.1
LibJPEG: 80
LibJasper: 1.900.1
LibKDE: 4.6.2 (4.6.2)
LibKExiv2: 2.0.0
LibKMap: 2.0.0
LibKdcraw: 2.0.0
LibLCMS: 119
LibPGF: 6.09.44 - internal library
LibPNG: 1.4.5
LibQt: 4.7.2
LibRaw: 0.13.4
LibTIFF: LIBTIFF, Version 3.9.5 Copyright (c) 1988-1996 Sam Leffler  
Copyright (c) 1991-1996 Silicon Graphics, Inc.
Marble Widget: 0.11.2 (Stable Release)
Parallelized demosaicing: Yes
Database backend: QSQLITE
LibGphoto2: 2.4.10.1
LibKface: 2.0.0
LibKipi: 1.2.0
LibOpenCV: 2.2.0
Libface: 0.2


Andi

--
Using Opera's revolutionary email client: http://www.opera.com/mail/
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

2011/4/22 Andi Clemens <[hidden email]>:

> Hi,
>
> since 3 months I get a lot of compile warnings especially with libjpeg:
>
> /home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:
> warning: enumeration value ‘JCS_EXT_RGB’ not handled in switch [-Wswitch]
> /home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:
> warning: enumeration value ‘JCS_EXT_RGBX’ not handled in switch [-Wswitch]
> /home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:
> warning: enumeration value ‘JCS_EXT_BGR’ not handled in switch [-Wswitch]
> /home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:
> warning: enumeration value ‘JCS_EXT_BGRX’ not handled in switch [-Wswitch]
> /home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:
> warning: enumeration value ‘JCS_EXT_XBGR’ not handled in switch [-Wswitch]
> /home/andi/Programmieren/KDE/digikam/libs/dimg/loaders/jpegloader.cpp:257:12:
> warning: enumeration value ‘JCS_EXT_XRGB’ not handled in switch [-Wswitch]
>
> Can we handle those new variables with
> default:break;

Yes, this must be the right way...

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