Why do you complicate the code so much?

synchronized( v ) {
    keys = new String[ v.children().keySet().size() ];
    keys = v.children().keySet().toArray( keys );
}

In this example, calling keySet simply makes no sense:

synchronized( v ) {
    keys = new String[ v.children().size() ];
    keys = v.children().keySet().toArray( keys );
}

instead of if( toBeClosed() == false )
you can use if( !toBeClosed() ) and so on.

These are temporary measures, or code style (to know whether it is possible to send a pull request) ?