Description

If the PVL-text was not decodable as UTF-8, then all characters at and after the non-UTF-8 character were not considered as part of the PVL-text for parsing. Characters encoded in latin-1 (ISO/IEC 8859-1) should be considered (and don't code as UTF-8), and now are.

Related Issue

Fixes #93

How Has This Been Tested?

make lint
make docs
make test-all

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

I have read the CONTRIBUTING document.
All existing tests passed.

Licensing:

This project is released under the LICENSE.

I claim copyrights on my contributions in this pull request, and I provide those contributions via this pull request under the same license terms that the pvl project uses.

Codecov Report

Merging #94 (6d15567) into main (1a1b41b) will increase coverage by 0.11%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   94.85%   94.97%   +0.11%     
==========================================
  Files          12       12              
  Lines        1925     1929       +4     
==========================================
+ Hits         1826     1832       +6     
+ Misses         99       97       -2

Impacted Files	Coverage Δ
pvl/__init__.py	`96.15% <100.00%> (+1.41%)`	:arrow_up:
pvl/grammar.py	`100.00% <100.00%> (ø)`
pvl/lexer.py	`99.23% <0.00%> (-0.77%)`	:arrow_down:
pvl/parser.py	`96.90% <0.00%> (+0.61%)`	:arrow_up:

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 1a1b41b...6d15567. Read the comment docs.

planetarypy / pvl

We were decoding UTF-8, but not latin-1. #94