You can make it one line, but I like the comment for when we try to figure
out why it's there in 3 years. :)
On Feb 7, 2014 8:15 PM, "jschneier" <notifications@github.com> wrote:

> In test/__init__.py:
>
> > @@ -21,7 +21,8 @@ def requires_network(test):
> >      """Helps you skip tests that require the network"""
> >
> >      def _is_unreachable_err(err):
> > -        return hasattr(err, 'errno') and err.errno == errno.ENETUNREACH
> > +        return hasattr(err, 'errno') and (err.errno == errno.ENETUNREACH or
>
> heh, I played around a little bit with in syntax and couldn't really
> settle on anything I liked. I like yours but 4 lines make me :(
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/shazow/urllib3/pull/335/files#r9562434>
> .
>