windelbouwman / ppci

A compiler for ARM, X86, MSP430, xtensa and more implemented in pure Python
https://ppci.readthedocs.io/en/latest/
BSD 2-Clause "Simplified" License
337 stars 36 forks source link

Fix issue89 #93

Closed tstreiff closed 4 years ago

tstreiff commented 4 years ago

In semantics.py/on_number(), the tests against the maximum values are changed from < to <= to allow the maximum value.

codecov-commenter commented 4 years ago

Codecov Report

Merging #93 into master will decrease coverage by 0.04%. The diff coverage is 20.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
- Coverage   78.55%   78.50%   -0.05%     
==========================================
  Files         344      349       +5     
  Lines       45143    45526     +383     
==========================================
+ Hits        35460    35741     +281     
- Misses       9683     9785     +102     
Impacted Files Coverage Δ
ppci/lang/c/semantics.py 83.65% <20.00%> (ø)
ppci/common.py 87.20% <0.00%> (ø)
ppci/__main__.py 0.00% <0.00%> (ø)
ppci/__init__.py 75.00% <0.00%> (ø)
ppci/runtime.py 0.00% <0.00%> (ø)
ppci/api.py 83.96% <0.00%> (ø)
ppci/lang/tools/lr.py 96.45% <0.00%> (+0.39%) :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 eab9f94...818072a. Read the comment docs.