Bug 40963

Summary: [EFL] Remove warning message during the EFLWebKit build
Product: WebKit Reporter: Gyuyoung Kim <gyuyoung.kim>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Trivial CC: lucas.de.marchi
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch none

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.