Hi Daniel, > On Nov 8, 2018, at 1:50 AM, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > So FWIW my point was that there's nothing that you can really > guarantee in InputSteam::skipFully() if a subclass implementation of > skip() uses negative number to e.g. signal abnormal conditions > (such as EOF?). And I wonder if that should warrant a disclaimer > in the API doc of InputStream::skipFully (maybe an @implSpec?)
I don’t know about having an explicit disclaimer; that seems a little strange to me. If the new method is defined in terms of skip() the I would think that it is implicit that misbehavior of the latter would cause problems. Thanks, Brian
