JuliaIO / Tar.jl

TAR files: create, list, extract them in pure Julia
MIT License
79 stars 19 forks source link

read_header_int: allow leading spaces #116

Closed StefanKarpinski closed 3 years ago

StefanKarpinski commented 3 years ago

There are a number of changes in this PR, but the main one is the fix for #111 which is to allow leading spaces in integer fields since some old tar programs emitted these fields that way (contrary to the spec). The other substantial change is improved checking and reporting of errors in headers. Here is the list of all changes:

codecov[bot] commented 3 years ago

Codecov Report

Merging #116 (45197a8) into master (343e923) will increase coverage by 0.79%. The diff coverage is 98.73%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
+ Coverage   96.05%   96.84%   +0.79%     
==========================================
  Files           4        4              
  Lines         684      729      +45     
==========================================
+ Hits          657      706      +49     
+ Misses         27       23       -4     
Impacted Files Coverage Δ
src/extract.jl 97.97% <98.66%> (+1.39%) :arrow_up:
src/create.jl 97.19% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 343e923...45197a8. Read the comment docs.