Oh ok, I tried having it has self.nrss but I think it was that I wanted
(now `max`) to be self.nrss, but for optional parameters setting it like
this isn't allowed so I scoped it inside.  There is probably a better way
to handle this.


On Fri, Dec 20, 2013 at 5:15 PM, Anthony Scopatz
<notifications@github.com>wrote:

> In pyne/mcnp.py:
>
> > @@ -191,19 +191,25 @@ def print_header(self):
> >
> >          return header_string
> >
> > -    def print_tracklist(self):
> > +    def print_tracklist(self,quit=None):
> > +
> > +        if quit is None:
> > +            quit = self.nrss
>
> quit normally is a boolean flag. Having it as none is a little strange.
> If it is supposed to be a max count then call it something like nmax or
> max_count.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/pyne/pyne/pull/229/files#r8514186>
> .
>