BitShares Forum
Other => Graveyard => MemoryCoin => Topic started by: dga on January 13, 2014, 12:52:51 pm
-
Hi, FreeTrade -
Line 94 of momentum.cpp has an incorrect comment:
//use last 4 bits of first cache as next location
4 should be 14 (or abstracted in terms of the other constants).
-
Not too much of a big deal, still functional. Although well done for pointing it out I'm sure he will appreciate it
-
Not too much of a big deal, still functional. Although well done for pointing it out I'm sure he will appreciate it
Yup, It's just a comment typo. :) It just makes it a little harder to understand the PoW function when reading the source.
-
Not too much of a big deal, still functional. Although well done for pointing it out I'm sure he will appreciate it
Yup, It's just a comment typo. :) It just makes it a little harder to understand the PoW function when reading the source.
the typo can affect how devs perceive the PoW
-
Not too much of a big deal, still functional. Although well done for pointing it out I'm sure he will appreciate it
Yup, It's just a comment typo. :) It just makes it a little harder to understand the PoW function when reading the source.
the typo can affect how devs perceive the PoW
Not too hard to realize the mistake, I can't speak for every dev but I use comments to see what does what, and then I read the code to see the how.