Bug 113953 - [EFL] --minimal build fails with error: expected `;' before `return'
Summary: [EFL] --minimal build fails with error: expected `;' before `return'
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Ed Bartosh
URL:
Keywords: EasyFix
Depends on:
Blocks:
 
Reported: 2013-04-04 13:22 PDT by Ed Bartosh
Modified: 2013-04-07 16:51 PDT (History)
7 users (show)

See Also:


Attachments
Patch (1.35 KB, patch)
2013-04-04 13:25 PDT, Ed Bartosh
no flags Details | Formatted Diff | Diff
Patch (1.33 KB, patch)
2013-04-07 13:25 PDT, Ed Bartosh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Bartosh 2013-04-04 13:22:49 PDT
/home/ed/git/wk/WebKit/Source/WebKit/efl/ewk/ewk_security_origin.cpp: In function `uint64_t ewk_security_origin_web_database_quota_get(const Ewk_Security_Origin*)':
/home/ed/git/wk/WebKit/Source/WebKit/efl/ewk/ewk_security_origin.cpp:85:5: error: expected `;' before `return'
/home/ed/git/wk/WebKit/Source/WebKit/efl/ewk/ewk_security_origin.cpp:87:1: error: no return statement in function returning non-void [-Werror=return-type]
/home/ed/git/wk/WebKit/Source/WebKit/efl/ewk/ewk_security_origin.cpp: At global scope:
/home/ed/git/wk/WebKit/Source/WebKit/efl/ewk/ewk_security_origin.cpp:89:6: warning: unused parameter `quota' [-Wunused-parameter]
cc1plus: all warnings being treated as errors
make[2]: *** [Source/WebKit/CMakeFiles/ewebkit.dir/efl/ewk/ewk_security_origin.cpp.o] Error 1
make[1]: *** [Source/WebKit/CMakeFiles/ewebkit.dir/all] Error 2
make: *** [all] Error 2
Comment 1 Ed Bartosh 2013-04-04 13:25:54 PDT
Created attachment 196512 [details]
Patch
Comment 2 Ed Bartosh 2013-04-07 13:25:37 PDT
Created attachment 196809 [details]
Patch

Rebased. Please, review.
Comment 3 Patrick R. Gansterer 2013-04-07 16:40:04 PDT
Committed r147885: <http://trac.webkit.org/changeset/147885>
Comment 4 Gyuyoung Kim 2013-04-07 16:51:35 PDT
Comment on attachment 196809 [details]
Patch

Clearing r?, cq?