This pull request introduces 8 alerts and fixes 19 when merging eb895a35bf24ad5b609f65df7b959f797b5f5b5e into 25d8f170f782be1468ae05efeabbca24f4284bf8 - view on LGTM.com
new alerts:
7 for FIXME comment
1 for Expression has no effect
fixed alerts:
6 for Expression has no effect
5 for FIXME comment
4 for Pointer overflow check
2 for Local variable hides global variable
1 for Unsigned comparison to zero
1 for Multiplication result converted to larger type
Looks like #92 actually did not include the reverted changes! (although they were on my branch)
So, this PR reverts the revert (and re-adds the build fixes on top).