PeterAlfreadLee commented on issue #83: COMPRESS-342 random access of 7z files URL: https://github.com/apache/commons-compress/pull/83#issuecomment-550886814 > The latest change may only be necessary when the content methods of `file` actually is `null`, which may not be the case when jumping backwards to a position that has already been reached by other means. But an additional check would mainly be an optimization. I agree. I will add the non-null check for the content methods, and the content methods will not be set when the content methods is not null. > Right now I'm not sure the code works in situations where we jump backwards inside the folder or whether we'd need to remove streams from `deferredBlockStreams` otherwise. Probably I'd need to write a test for that :-) I agree. I forgot the situations that we may jump backwards inside the folder. It seems I can only reopen the folder and skip the previous files again. I will modify both of these 2 and push my code today.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
