You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
if with m.Case(): is going to be the way to get a default case, that needs to be documented somewhere, I was not aware that nmigen supported default cases at all.
I personally prefer having a separate Default function, since that is more explicit.
Ah, ok. I had a suspicion that default wasn't actually supported in nMigen (since I couldn't find "default" in the source) so I fell back to the Migen docs. So now you fixed two issues :D
It seems to be more consistent with most other commonly-known languages to have the syntax for a default case look like this:
than this:
The text was updated successfully, but these errors were encountered: