hdl.ir: allow disabling UnusedElaboratable warning in file scope.
authorwhitequark <whitequark@whitequark.org>
Sat, 26 Oct 2019 05:34:00 +0000 (05:34 +0000)
committerwhitequark <whitequark@whitequark.org>
Sat, 26 Oct 2019 06:17:14 +0000 (06:17 +0000)
commit9786d0c0e3684ff9e4ebcd923fb7db1067c0635a
tree1e001051d25d571f709828bc1941245ebea9bc4d
parent8b05b28f5a7c2ad3613fd876edd16204fd6af2cb
hdl.ir: allow disabling UnusedElaboratable warning in file scope.

This warning is usually quite handy, but is problematic in tests:
although it can be suppressed by using Fragment.get on elaboratable,
that is not always possible, in particular when writing tests for
exceptions raised by __init__, e.g.:

    def test_wrong_csr_bus(self):
        with self.assertRaisesRegex(ValueError, r"blah blah"):
            WishboneCSRBridge(csr_bus=object())

In theory, it should be possible to suppress warnings per-module
and even per-line using code such as:

    import re, warnings
    from nmigen.hdl.ir import UnusedElaboratable
    warnings.filterwarnings("ignore", category=UnusedElaboratable,
                            module=re.escape(__name__))

Unfortunately, not only is this code quite convoluted, but it also
does not actually work; we are using warnings.warn_explicit() because
we collect source locations on our own, but it requires the caller
to extract the __warningregistry__ dictionary from module globals,
or warning suppression would not work. Not only is this not feasible
in most diagnostic sites in nMigen, but also I never got it to work
anyway, even when passing all of module, registry, and module_globals
to warn_explicit().

Instead, use a magic comment at the start of a file to do this job,
which might not be elegant but is simple and practical. For now,
only UnusedElaboratable can be suppressed with it, but in future,
other linter parameters may become tweakable this way.
nmigen/_utils.py
nmigen/hdl/ir.py