fix `ret` flag initialization

graphitemaster / incbin

Include binary files in C/C++

The Unlicense

966 stars 90 forks source link

fix `ret` flag initialization #47

Closed esaulenka closed 3 years ago

esaulenka commented 3 years ago

I found few errors in incbin utility. 1) ret flag not initialized to zero at startup. Therefore, utility can never terminate correctly. I dont know, why you not found this bug before: it always deletes output file in last step. This has also mentioned here: https://github.com/graphitemaster/incbin/issues/39

2) ret flag is not set, if utility can not read input binary.

kervinpierre commented 3 years ago

why not accept this trivial patch?