Add a clarifying comment to the memory layout navigation. (Inspired by Coverity CID 100022)

Add a clarifying comment to the memory layout navigation. (Inspired by Coverity CID 100022)

R=tommi@chromium.org
BUG=None
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148412

[tommi (sloooow) - chröme, 2012-07-25 19:04:05]

lgtm

https://chromiumcodereview.appspot.com/10826014/diff/1/courgette/memory_alloc...
File courgette/memory_allocator.cc (right):

https://chromiumcodereview.appspot.com/10826014/diff/1/courgette/memory_alloc...
courgette/memory_allocator.cc:139: // go past it. (See Initialize()).
Ah, nice!  That's not at all clear from the code.
nit: mapping_.view()

[Greg Billock, 2012-07-25 20:12:04]

https://chromiumcodereview.appspot.com/10826014/diff/1/courgette/memory_alloc...
File courgette/memory_allocator.cc (right):

https://chromiumcodereview.appspot.com/10826014/diff/1/courgette/memory_alloc...
courgette/memory_allocator.cc:139: // go past it. (See Initialize()).
On 2012/07/25 19:04:05, tommi wrote:
> Ah, nice!  That's not at all clear from the code.
> nit: mapping_.view()

Done.

[commit-bot: I haz the power, 2012-07-25 20:12:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/10826014/5001

[commit-bot: I haz the power, 2012-07-25 21:26:14]

Change committed as 148412