TheDiamondProject / Graphite

MIT License
16 stars 6 forks source link

Classic file verification #42

Closed andrews05 closed 3 years ago

andrews05 commented 3 years ago

On rare occasion, the second preamble in a resource file may be empty. This PR changes the verification to ignore zero values in the second preamble. Additionally, it verifies that the first preamble is not zero, which would otherwise be successfully parsed with an odd result.

(Fixes #32)

andrews05 commented 3 years ago

@tjhancocks Have you had a chance to look at this? I've just noticed #32, which this PR will fix.

tjhancocks commented 3 years ago

Sorry been quite ill through October and changing jobs, so only just had chance to see this.