>
> !entry->writable && (inUser || cr0.wp) is used here and in the if below. This 
> would be easier to read if that was in a temporary variable, maybe called 
> badWrite. Nate would probably yell at me that it should be bad_write because 
> it's local. He'd be right, but that would be inconsistent with the existing 
> code.
>
>
If the inconsistency bothers you, then first fix all the other code that's
not following the style :-).

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

Reply via email to