Skip to content

chore: remove dead mem.make code and stale mem.size_of comment - #2513

Merged
SchoolyB merged 1 commit into
grayscale-lang:bugfixes/september-1-2026from
coincoin57660-byte:chore/remove-dead-mem-code
Aug 27, 2026
Merged

chore: remove dead mem.make code and stale mem.size_of comment#2513
SchoolyB merged 1 commit into
grayscale-lang:bugfixes/september-1-2026from
coincoin57660-byte:chore/remove-dead-mem-code

Conversation

@coincoin57660-byte

@coincoin57660-byte coincoin57660-byte commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Related issue

Closes #2479

Summary

  • Remove the dead mem.make typechecker branch.
  • Remove the stale mem.size_of comment.

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Tests
  • Documentation
  • CI/Build

Breaking changes

None.

Checklist

  • make build compiles with zero warnings
  • Tests added/updated (unit, integration, or both as appropriate)
  • No @ before module names in any text (it tags GitHub users)
  • Added yourself to the Contributors: section of the header of each source file you changed

If adding user-facing stdlib/builtin/type

  • C implementation in grayc/src/stdlib/<module>.h and .c (with @man block)
  • Typechecker wired: return type, stdlib_arg_table[], stdlib_arg_type_table[], _using_funcs[]
  • Codegen wired: emit_<module>_call()
  • STANDARD.md updated
  • ./scripts/generate_stdlib_man.sh run and output committed

@coincoin57660-byte

Copy link
Copy Markdown
Contributor Author

Did I do something wrong? The issue says, ‘Two leftovers reference mem module functions that do not exist. Neither affects behavior — both are cleanup.’
So...

@SchoolyB SchoolyB added the PR: Ready This PR is ready pull request is ready to merge label Aug 27, 2026
@SchoolyB
SchoolyB changed the base branch from main to bugfixes/september-1-2026 August 27, 2026 08:00
@SchoolyB
SchoolyB merged commit 8418770 into grayscale-lang:bugfixes/september-1-2026 Aug 27, 2026
4 of 7 checks passed
@coincoin57660-byte
coincoin57660-byte deleted the chore/remove-dead-mem-code branch August 27, 2026 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Ready This PR is ready pull request is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: remove dead mem.make code and stale mem.size_of comment

2 participants