Well, as `skip_header` is a number of lines, I don't really see anything 
particular magical about a `skip_header=-1`. Plus, range(-1) == [], while 
range("comments") raises a TypeError. And then you'd have to figure why the 
exception was raised.

-- 
Pierre GM


On Monday, July 16, 2012 at 21:56 , Nathaniel Smith wrote:

> On Mon, Jul 16, 2012 at 8:06 PM, Paul Natsuo Kishimoto
> <m...@paul.kishimoto.name (mailto:m...@paul.kishimoto.name)> wrote:
> > I've implemented this feature with skip_header=-1 as suggested by
> > Pierre, and in doing so removed the regression. TravisBot seems to like
> > it: https://github.com/numpy/numpy/pull/351
> > 
> 
> 
> Can we please not use weird magic values like this? This isn't C.
> skip_header="comments" would work just as well and be far more
> self-explanatory...
> 
> -n
> _______________________________________________
> NumPy-Discussion mailing list
> NumPy-Discussion@scipy.org (mailto:NumPy-Discussion@scipy.org)
> http://mail.scipy.org/mailman/listinfo/numpy-discussion
> 
> 


_______________________________________________
NumPy-Discussion mailing list
NumPy-Discussion@scipy.org
http://mail.scipy.org/mailman/listinfo/numpy-discussion

Reply via email to