I'm happy to get rid of `locked`, but will leave a note in there for anyone
who tries to add post-building tree expansion. Comments coming.


On 25 January 2014 02:17, Peter Prettenhofer <notifications@github.com>wrote:

>  In sklearn/tree/_tree.pyx:
>
> >
> >      # XXX using (size_t)(-1) is ugly, but SIZE_MAX is not available in C89
> >      # (i.e., older MSVC).
> >      cdef int _resize_c(self, SIZE_t capacity=<SIZE_t>(-1)) nogil:
> >          """Guts of _resize. Returns 0 for success, -1 for error."""
> > +        if self.locked:
> > +            return 1
>
> if 0 is success and -1 is error -- what is 1?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/2790/files#r9150939>
> .
>