[Digikam-devel] [Bug 157314] Zoom-slider has no steps

Gilles Caulier caulier.gilles at gmail.com
Wed Apr 23 07:46:47 BST 2008


------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=157314         




------- Additional Comments From caulier.gilles gmail com  2008-04-23 08:46 -------
Arnd,

Patch is fine for me. Just remove the #include <stdlib.h> from top of imagepreview.cpp...

C Ansi header must be limited if possible to improve portability.

Another tip : if you really need to use C Ansi header, i recommend to always use C++ header instead, we it's possible (it not always the case). For ex.:

#include <math.h> ==> #include <cmath>

C++ header don't use .h extension and generally add 'c' char on front of file name.

Note than not all C Ansi header file have been converter to C++ like (for Ex. #include <stdint.h>). In this case the C Ansi header need to be wrapped around "extern "C" {...}"...

Gilles



More information about the Digikam-devel mailing list