Skip to content

Conversation

@miurahr
Copy link
Owner

@miurahr miurahr commented Feb 1, 2020

  • Drop last_alternative and loop depends on it.
  • Current Folder class assume last_alternative flag,
    which does not exist in LZMA SDK 16.00 in p7zip.

Signed-off-by: Hiroshi Miura [email protected]

- Drop last_alternative and loop depends on it.
- Current Folder class assume last_alternative flag,
  which does not exist in LZMA SDK 16.00 in p7zip.

Signed-off-by: Hiroshi Miura <[email protected]>
@codecov
Copy link

codecov bot commented Feb 1, 2020

Codecov Report

Merging #62 into master will decrease coverage by 0.02%.
The diff coverage is 75.86%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   83.44%   83.42%   -0.03%     
==========================================
  Files           8        8              
  Lines        2169     2166       -3     
  Branches      352      351       -1     
==========================================
- Hits         1810     1807       -3     
- Misses        231      232       +1     
+ Partials      128      127       -1
Flag Coverage Δ
#linux 82.73% <75.86%> (-0.03%) ⬇️
#mac 82.54% <75.86%> (-0.03%) ⬇️
#py35 79.29% <75.86%> (-0.03%) ⬇️
#py36 83.18% <75.86%> (-0.03%) ⬇️
#py37 83.18% <75.86%> (-0.03%) ⬇️
#py38 83.22% <75.86%> (-0.02%) ⬇️
#windows 80.79% <75.86%> (-0.03%) ⬇️
Impacted Files Coverage Δ
py7zr/archiveinfo.py 83.11% <75.86%> (-0.07%) ⬇️

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 3319938...cd70974. Read the comment docs.

@miurahr miurahr merged commit ca9de59 into master Feb 1, 2020
@miurahr miurahr deleted the patch-folder-codecs-metadata branch February 3, 2020 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants