RESOLVED DUPLICATE of bug 40965 40963
[EFL] Remove warning message during the EFLWebKit build 
https://bugs.webkit.org/show_bug.cgi?id=40963
Summary [EFL] Remove warning message during the EFLWebKit build
Gyuyoung Kim
Reported 2010-06-21 19:20:18 PDT
There are warning message when EFLWebKit is built. [100%] Building CXX object WebKit/CMakeFiles/ewebkit.dir/efl/ewk/ewk_view.cpp.o [100%] Building C object WebKit/CMakeFiles/ewebkit.dir/efl/ewk/ewk_view_single.c.o /home/gyuyoung/webkit/WebKit-EWebLauncher/WebKit/efl/ewk/ewk_view.cpp: In function ‘uint64_t ewk_view_exceeded_database_quota(Evas_Object*, Evas_Object*, const char*, uint64_t, uint64_t)’: /home/gyuyoung/webkit/WebKit-EWebLauncher/WebKit/efl/ewk/ewk_view.cpp:3452: warning: format ‘%lu’ expects type ‘long unsigned int’, but argument 7 has type ‘uint64_t’ /home/gyuyoung/webkit/WebKit-EWebLauncher/WebKit/efl/ewk/ewk_view.cpp:3452: warning: format ‘%lu’ expects type ‘long unsigned int’, but argument 8 has type ‘uint64_t’ Linking CXX shared library libewebkit.so [100%] Building C object WebKit/CMakeFiles/ewebkit.dir/efl/ewk/ewk_view_single.c.o /home/gyuyoung/webkit/WebKit-EWebLauncher/WebKit/efl/ewk/ewk_view.cpp: In function ‘uint64_t ewk_view_exceeded_database_quota(Evas_Object*, Evas_Object*, const char*, uint64_t, uint64_t)’: /home/gyuyoung/webkit/WebKit-EWebLauncher/WebKit/efl/ewk/ewk_view.cpp:3452: warning: format ‘%u’ expects type ‘unsigned int’, but argument 7 has type ‘uint64_t’ /home/gyuyoung/webkit/WebKit-EWebLauncher/WebKit/efl/ewk/ewk_view.cpp:3452: warning: format ‘%u’ expects type ‘unsigned int’, but argument 8 has type ‘uint64_t’
Attachments
Patch (1.09 KB, patch)
2010-06-21 19:24 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Gyuyoung Kim
Comment 1 2010-06-21 19:24:14 PDT
Created attachment 59324 [details] Patch It seems that uint64_t is defined as "unsigned long long". So, I change the %lu with %llu.
Gyuyoung Kim
Comment 2 2010-06-21 19:45:35 PDT
Opps, Lukas already made a bug for this. *** This bug has been marked as a duplicate of bug 40965 ***
Lucas De Marchi
Comment 3 2010-06-21 20:21:00 PDT
(In reply to comment #1) > Created an attachment (id=59324) [details] > Patch > > It seems that uint64_t is defined as "unsigned long long". So, I change the %lu with %llu. %llu will be wrong in 64-bit systems. To improve portability, we will need inttypes.h.
Note You need to log in before you can comment on or make changes to this bug.