Skip to content

[TM-374] Removing `NStar`

Description

Currently Notes are represented as either NStar or actual notes. The need to handle special ctor makes code too complicated at places and prevents some features from being implemented.

This optimization has been introduced to avoid O(n) types comparison (which happens, potentially, in typechecking of every single instruction).

Related issue(s)

https://issues.serokell.io/issue/TM-374

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:

Stylistic guide (mandatory)

Edited by Андреев Кирилл

Merge request reports