Skip to content

[Chore] Remove unneeded hack working around pattern match checker

Note: The fixup commit is for the ease of review, the first commit only removes the nested case and changes indentation. Unfortunately, this completely breaks the diff. Missing patterns are thus added as a fixup commit to make it more obvious what changed.

Description

Problem: At some point in the past, GHC couldn't handle large pattern matches. Now it can. However, we still have this hack, and hence don't statically check for the exhaustiveness of our patterns.

Solution: Remove the hack, let GHC check all the patterns.

Related issue(s)

None.

Checklist for your Merge Request

Related changes (conditional)

  • Tests (see short guidelines)

    • If I added new functionality, I added tests covering it.
    • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • Documentation

    • I checked whether I should update the docs and did so if necessary:
    • I updated changelog files of all affected packages released to Hackage if my changes are externally visible.

N/A

Stylistic guide (mandatory)

Edited by Nikolay Yakimov

Merge request reports