[COMPRESS-64] Are the public finish() methods ArchiveOutputStream implementations necessary and safe?

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0
• Fix Version/s: 1.0
• Assignee: Unassigned
• Reporter: Sebb
• Created: 28/Mar/09 14:12
• Updated: 02/May/23 17:11
• Resolved: 23/Apr/09 05:34

Description

Some of the ArchiveOutputStream implementations have public finish() methods. These are currently only called from the close() methods.

Seems to me that there is no need to allow the finish() methods to be called externally, and the user can corrupt the output if they do.

Surely the close() method is all that is needed?