Re:Zcoin implementation bug enabled attacker to create over ...
by WalterBright on Tuesday March 07, 2017 @01:48AM
(#13808728)
We've modified the dlang Github tester to run coverage tests over Pull Requests, and uncovered changes get marked with a scarlet bar. While some changes can't be tested with a test suite, the result has been a marked improvement in the quality of pull requests (including mine) because it is embarrassing to submit untested code.
Re:Zcoin implementation bug enabled attacker to create over ...
by unbeli on Tuesday March 07, 2017 @04:42AM
(#13809299)
I think that won't even compile due to an accidentally pasted URL here:
https://github.com/zcoinofficial/zcoin/commit/0359bcb2ead7fe...
https://github.com/zcoinofficial/zcoin/commit/0359bcb2ead7fe...
This indicates that the code in question is not covered by tests. I work on a database, for which correctness is paramount, so it's unnerving to see any code fixes not associated with test additions or changes. I would have thought that cryptocurrency had similar standards.