]> Cypherpunks repositories - gostls13.git/commit
archive/tar: remove writeHeader and writePAXHeaderLegacy
authorJoe Tsai <joetsai@digital-static.net>
Mon, 14 Aug 2017 21:59:18 +0000 (14:59 -0700)
committerJoe Tsai <thebrokentoaster@gmail.com>
Mon, 14 Aug 2017 23:29:25 +0000 (23:29 +0000)
commit694875cbf27a70ce2f9b147a30a644380a82860e
tree68ef5175c17ecd6657cf51500f492737cad45e24
parentffd9810e5966c8ad972f429866808b73bbefd8bc
archive/tar: remove writeHeader and writePAXHeaderLegacy

Previous CLs (CL/54970, CL55231, and CL/55237) re-implemented tar.Writer
entirely using specialized methods (writeUSTARHeader, writePAXHeader,
and writeGNUHeader) allowing tar.Writer to entirely side-step the broken
and buggy logic in writeHeader.

Since writeHeader and writePAXHeaderLegacy is now dead-code,
we can delete them.

One minor change is that we call Writer.Flush at the start of WriteHeader.
This used to be performed by writeHeader, but doing so in WriteHeader
ensures each of the specialized methods can benefit from its effect.

Fixes #17665
Fixes #12594

Change-Id: Iff2ef8e7310d40ac5484d2f8852fc5df25201426
Reviewed-on: https://go-review.googlesource.com/55550
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
src/archive/tar/writer.go