- a race condition: the reader could be closed between the time you get it and the time you try to call reopen().
Ah, because of no incref...
- descriptor leak: you pass closeReader=false, but no one else will close this reader.
Dumb mistake here - made a private method public just so I could pass true and then still passed false...
>>Also, I think that any test that expects the reader to be different should be changed.
Alright, easy enough, just two I think: elevation and function tests, using the reader as a key in a map or something.
If thats all for the reopen, I've got that looking good I think, just have to take care of the tests.