feature-AST-using-arena #8

laentropia · 2026-04-23T12:34:04-06:00

laentropia commented

2026-04-23 12:34:04 -06:00

arena implementation

laentropia added 2 commits 2026-04-23 12:34:05 -06:00

rework: AST now uses an arena for allocation e4ec102cb9

For now it works but i dont really like that i use ParseResult, i mean
is necessary but i think i will try to make it cleaner so that i can
just directly use like parse and pass tath into evaluate, that would
require to move the main evaluate funciton into evaluate_tree or
something  and evaluate takes the arena, uses evaluate_tree and frees
the arena, will try that the next commit but for now this version works
perfectly.

fix/refactor: Modified evaluate and changed it to evaluate_tree 7ad4eba123

So i did what the last commit said, also fixed parse_expr because it was
still using malloc for allocating new nodes so i made it use arena_alloc
like it should, did the very first tests so it's all good, i think is
readdy to merge.

laentropia merged commit f50546bd07 into main

2026-04-23 12:34:30 -06:00

laentropia referenced this issue from a commit

2026-04-23 12:34:30 -06:00

Merge pull request 'feature-AST-using-arena' (#8) from feature-AST-using-arena into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: laentropia/Calculator#8