search

pawitp / protobuf-decoder

JavaScript-based web UI to decode ad-hoc Protobuf data
https://protobuf-decoder.netlify.app/
MIT License
392 stars 91 forks source link

Fix fixed32 decoding: protobuf stores as little-endian #8

Closed pawitp closed 5 years ago

pawitp commented 5 years ago

Also refactor decoder to separate file for testability.

codecov[bot] commented 5 years ago

Codecov Report

Merging #8 into master will increase coverage by 4.34%. The diff coverage is 76.92%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   52.89%   57.24%   +4.34%     
==========================================
  Files           6        7       +1     
  Lines         138      145       +7     
  Branches       29       29              
==========================================
+ Hits           73       83      +10     
+ Misses         60       58       -2     
+ Partials        5        4       -1
Impacted Files Coverage Δ
src/ProtobufPart.js 0% <0%> (ø) :arrow_up:
src/protobufPartDecoder.js 100% <100%> (ø)

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 dc4eb5b...212562c. Read the comment docs.