WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
112365
[BlackBerry] Enable USE_SYSTEM_MALLOC by default
https://bugs.webkit.org/show_bug.cgi?id=112365
Summary
[BlackBerry] Enable USE_SYSTEM_MALLOC by default
Alberto Garcia
Reported
2013-03-14 10:50:07 PDT
I'm getting lots of these otherwise: In file included from /home/berto/devel/code/webkit/Source/WTF/config.h:30:0, from /home/berto/devel/code/webkit/Source/WTF/wtf/HashTable.cpp:20: /home/berto/devel/code/webkit/Source/WTF/wtf/Platform.h:501:0: warning: "USE_SYSTEM_MALLOC" redefined [enabled by default] /home/berto/devel/code/webkit/WebKitBuild/armle-v7/Release/cmakeconfig.h:131:0: note: this is the location of the previous definition
Attachments
Patch
(1.29 KB, patch)
2013-03-14 10:52 PDT
,
Alberto Garcia
no flags
Details
Formatted Diff
Diff
Patch
(2.43 KB, patch)
2013-03-18 10:44 PDT
,
Alberto Garcia
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Alberto Garcia
Comment 1
2013-03-14 10:52:23 PDT
Created
attachment 193150
[details]
Patch
Alberto Garcia
Comment 2
2013-03-18 10:44:43 PDT
Created
attachment 193600
[details]
Patch Since there's the build option to enable/disable it, we don't need to hardcode its value in wtf/Platform.h
Benjamin Poulain
Comment 3
2013-03-18 12:20:03 PDT
Comment on
attachment 193600
[details]
Patch I guess you only build with build-webkit?
Alberto Garcia
Comment 4
2013-03-19 03:03:26 PDT
(In reply to
comment #3
)
> (From update of
attachment 193600
[details]
) > I guess you only build with build-webkit?
Exactly
WebKit Review Bot
Comment 5
2013-03-19 10:54:58 PDT
Comment on
attachment 193600
[details]
Patch Clearing flags on attachment: 193600 Committed
r146222
: <
http://trac.webkit.org/changeset/146222
>
WebKit Review Bot
Comment 6
2013-03-19 10:55:01 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug