Skip to content

Analyser: simplify conversion checker#506

Merged
bvdberg merged 1 commit intoc2lang:masterfrom
chqrlie:conversions
Apr 23, 2026
Merged

Analyser: simplify conversion checker#506
bvdberg merged 1 commit intoc2lang:masterfrom
chqrlie:conversions

Conversation

@chqrlie
Copy link
Copy Markdown
Contributor

@chqrlie chqrlie commented Apr 17, 2026

  • rename ambiguous objects and type functions:
  • Analyser.check -> Analyser.checkModule
  • conversion_checker.Check.check -> conversion_checker.Check.checkAssign
  • ma.checker -> ma.conv_checker
  • remove bogus absolete code in Checker.checkAssign
  • simplify conversion switches: factorize identical cases

@chqrlie chqrlie force-pushed the conversions branch 6 times, most recently from 9eea870 to c3dd607 Compare April 22, 2026 10:35
* rename ambiguous objects and type functions:
 - `Analyser.check` -> `Analyser.checkModule`
 - `conversion_checker.Check.check` -> `conversion_checker.Check.checkAssign`
 - `ma.checker` -> `ma.conv_checker`
* remove bogus absolete code in `Checker.checkAssign`
* simplify conversion switches: factorize identical cases
* simplify `get_common_arithmetic_type` and `usual_arithmetic_conversion`: use
  use `BuiltinKind` as index and matrix type
* change `BuiltKind` enum: move `Bool` in front of other types so `ISize`
  and `USize` at at the end
@bvdberg bvdberg merged commit f6ae333 into c2lang:master Apr 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants