http://codereview.chromium.org/8115014/diff/1/src/platform-linux.cc File src/platform-linux.cc (right):
http://codereview.chromium.org/8115014/diff/1/src/platform-linux.cc#newcode459 src/platform-linux.cc:459: mmap(GetRandomMmapAddr(), size, PROT_READ | PROT_WRITE, On 2011/10/04 09:18:33, Vyacheslav Egorov wrote:
preferred formatting is one arguments per line
Done. http://codereview.chromium.org/8115014/diff/1/src/platform-linux.cc#newcode475 src/platform-linux.cc:475: mmap(GetRandomMmapAddr(), size, PROT_READ | PROT_WRITE, On 2011/10/04 09:18:33, Vyacheslav Egorov wrote:
ditto
Done. http://codereview.chromium.org/8115014/diff/1/src/platform-linux.cc#newcode561 src/platform-linux.cc:561: void* addr = mmap(GetRandomMmapAddr(), size, PROT_READ | PROT_EXEC, On 2011/10/04 09:18:33, Vyacheslav Egorov wrote:
ditto
Done. http://codereview.chromium.org/8115014/diff/1/src/platform-macos.cc File src/platform-macos.cc (right): http://codereview.chromium.org/8115014/diff/1/src/platform-macos.cc#newcode95 src/platform-macos.cc:95: static void* GetRandomMmapAddr() { On 2011/10/04 09:18:33, Vyacheslav Egorov wrote:
consider moving somewhere (e.g. platfrom-posix.cc) to avoid
duplication.
platform can define the range of unpopular addresses which shared
implementation
would use.
Done. http://codereview.chromium.org/8115014/diff/1/src/platform-macos.cc#newcode169 src/platform-macos.cc:169: void* mbase = mmap(GetRandomMmapAddr(), msize, prot, On 2011/10/04 09:18:33, Vyacheslav Egorov wrote:
formatting: one argument per line
Done. http://codereview.chromium.org/8115014/diff/1/src/platform-macos.cc#newcode228 src/platform-macos.cc:228: mmap(GetRandomMmapAddr(), size, PROT_READ | PROT_WRITE, On 2011/10/04 09:18:33, Vyacheslav Egorov wrote:
formatting: one argument per line
Done. http://codereview.chromium.org/8115014/diff/1/src/platform-macos.cc#newcode244 src/platform-macos.cc:244: mmap(GetRandomMmapAddr(), size, PROT_READ | PROT_WRITE, On 2011/10/04 09:18:33, Vyacheslav Egorov wrote:
formatting: one argument per line
Done. http://codereview.chromium.org/8115014/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
