if m.If(...): is silently dropped

[nmigen-issue-migration]

Issue by Stary2001
Monday Dec 23, 2019 at 07:50 GMT
Originally opened as m-labs/nmigen#284

---

In some nMigen code I had if m.If(...) as a typo of with m.If(...): - this gets silently dropped.

<whitequark> Stary: the problem is that you're doing `if m.If():`
<whitequark> it should be `with m.If():`
<whitequark> `m.If()` itself just returns `None`, so the whole conditional is just ignored
<whitequark> I consider this a usability bug; please report an issue so I can make sure no one ever gets confused by this again