[Webkit-unassigned] [Bug 85063] [BlackBerry] Add low memory check in ExecutableAllocator::underMemoryPressure() for BlackBerry platform

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=85063





--- Comment #7 from Yong Li <yong.li.webkit at gmail.com>  2012-04-27 10:13:06 PST ---
(From update of attachment 139216)
View in context: https://bugs.webkit.org/attachment.cgi?id=139216&action=review

Overall it is a much better solution.

> Source/JavaScriptCore/ChangeLog:3
> +        [BlackBerry] Add low memory check in ExecutableAllocator::underMemoryPressure() for BlackBerry platform

This line is no longer accurate and should be changed?

> Source/WTF/wtf/SystemMemoryStatus.h:34
> +#define isMemoryLow BlackBerry::Platform::isMemoryLow

If we must use macro, I would use #define isMemoryLow() BlackBerry::Platform::isMemoryLow.

But I would rather use inline function in "namespace WTF".

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.