On Fri, 15 Oct 2010 21:14:39 -0700, Steve Reinhardt <[email protected]> wrote:
On Fri, Oct 15, 2010 at 8:43 PM, Ali Saidi <[email protected]> wrote:


On 2010-10-15 17:49:29, Steve Reinhardt wrote:
> I realize these names predate this patch, but your changes make it evident that cachedSize and cachedStart are actually kind of dumb names, so this might be a good time to fix that.  Since these are really just member variables that have accessors, shouldn't they be named _size and _start?
>

They have accessors? We could add some, but I don't think they have any.

http://repo.m5sim.org/m5/file/a7cb52c76262/src/mem/physical.hh#l156
Fixed...

Ali


_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to