is it the comparison == where you'd want an assignment =?
Took me almost 20 minutes to find the bug in this code.. https://github.com/cliffordwolf/yosys/commit/eb7b18e897ac908e960bee6c976f744043590881 …pic.twitter.com/lbGP1gkRbC
-
-
-
yep. see the link in my tweet.
End of conversation
New conversation -
-
-
Thanks. Twitter will use this to make your timeline better. UndoUndo
-
-
-
probably static analizer like cppcheck could help for such gotchas
-
Usually this should trigger a compiler warning. I think it didn't in this case because "==" is an overloaded operator.
End of conversation
New conversation -
-
-
I won't hold it against you :)
Thanks. Twitter will use this to make your timeline better. UndoUndo
-
-
-
Are you running static analysis on the Yosys codebase right now? If not, that would probably have caught it.
-
What static analysis tool (s) do you recommend?
@oe1cxw - Show replies
New conversation -
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.