Bug 40963 - [EFL] Remove warning message during the EFLWebKit build
Summary: [EFL] Remove warning message during the EFLWebKit build
Status: RESOLVED DUPLICATE of bug 40965
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Trivial
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-21 19:20 PDT by Gyuyoung Kim
Modified: 2010-06-21 21:32 PDT (History)
1 user (show)

See Also:


Attachments
Patch (1.09 KB, patch)
2010-06-21 19:24 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gyuyoung Kim 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’
Comment 1 Gyuyoung Kim 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.
Comment 2 Gyuyoung Kim 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 ***
Comment 3 Lucas De Marchi 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.