Not arguing that it is wrong., cuz it is.  But not fixing in this pass.

Matthew

On Jul 19, 2013, at 4:06 PM, Dave Parfitt <notifications@github.com> wrote:

> In util/env_posix.cc:
> 
> >        else
> >        {
> > -          Env::Default()->Schedule(&BGFileUnmapper2, ptr, 4);
> > +          if (and_close)
> > +              Env::Default()->Schedule(&BGFileCloser2, ptr, 4);
> looks like that 4 refers to a specific state, can that be constant?
> 
> —
> Reply to this email directly or view it on GitHub.
>