Re: [Rails-devel] Patchfiles

[Brett L. <wak...@gm...>]

On Tue, 2008-06-03 at 23:32 +0200, Freek Dijkstra wrote:
> Brett Lentz wrote:
> 
> >> +        if (tokenMode) {
> >> +            // nothing to show
> >> +        } else {
> > 
> > Is there a reason you didn't use "if (!tokenMode)" ?
> 
> No. I copied it, and forgot to check later (I dislike the implicit 
> assumption that there are only two modes, but never felt it was 
> important enough to improve).
> 
> Regards,
> Freek
> 

Ok.  I've committed your patch, and just simplified the statement.


---Brett.