laentropia/Calculator

Fork 0

refactor-error-handling #12

Merged

laentropia merged 11 commits from refactor-error-handling into main

2026-05-14 01:05:55 +00:00

laentropia commented

2026-05-14 01:05:23 +00:00

Owner

Reworked everything to work as expected, jus evaluator needs error handlig wich is a bit crazy given that that was the porpuse of this rework but it ended up being a rework of everything.

the necessary testing and better handlig of errors as output will be taken into account int the further testing issue, (i just want to be done for now)

Reworked everything to work as expected, jus evaluator needs error handlig wich is a bit crazy given that that was the porpuse of this rework but it ended up being a rework of everything. the necessary testing and better handlig of errors as output will be taken into account int the further testing issue, (i just want to be done for now)

laentropia added this to the First working version of calculator milestone

2026-05-14 01:05:23 +00:00

laentropia self-assigned this

2026-05-14 01:05:23 +00:00

laentropia added 11 commits

2026-05-14 01:05:24 +00:00

refactor: delete ASTNode, add Token to lexer 2a73f5f9d6

So, total refactor, now we serious. I feel ASTNode was feeling very
bloated so we need to rewrite and adapt everything, by now lets get the
lexer working again, is already well written for me at least.

refactor: adapted lexer to work with new tokens f3373123e1

Now its fine, the code i find it clear if one just sits down to read it
for a moment, next is the parser that REALLY needs reworking and a few
helper structs.

refactor: changed parser.h, added Node 80e05a9acf

So just added node back but now clearly separated by tokens and nodes of
the AST as it should be, now real rework the mess that is the parser

refactor: All of parser.c 542a94ef81

DAMN, it wasn't that difficult, just bothers me a bit the part that
checks if both lbp and rbp of the infix are valid, like i do validation
twice but is fine i guess, maybe using an else?, i'll see if i change
it, for now i need to change the evaluator

refactor: evaluator incomplete. efa0e3bacd

SO, i forgot to implement nud and led correctly and the parser cant tell
apart from - as unary and - as binary (+ as well), i need to correct
that, move Node * to TreeResult so to use NodeResult with nud and led

refactor: moved NodeResult to TreeResult 90c426f3a4

addtition: nud and led token to node distinction 70ab06964c

I think i should instead have a nud and led function i guess, may do
that next

refactor: nud and led have differetn responasblires 6294121e91

So now nud and led do what they were supposed to do i guess, now i
thinks is just adjusting infix and postfix and all bd funcions to act on
operator instead.

refactor: bp funtions take tokens now b56a368244

necessary for cleannes

refactor: evaluator separated into evaluate bin and un 3ec73559ee

Is nicer this way, also made it more beautiful to look at and therefor
to understand.

fix: tests and main ab791dbc9b

laentropia added this to the (deleted) project

2026-05-14 01:05:24 +00:00

laentropia merged commit 9ea1da549f into main

2026-05-14 01:05:55 +00:00

laentropia deleted branch refactor-error-handling

2026-05-14 01:05:55 +00:00

laentropia referenced this pull request from a commit

2026-05-14 01:05:55 +00:00

Merge pull request 'refactor-error-handling' (#12) from refactor-error-handling into main

No reviewers

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

laentropia/Calculator!12

No description provided.

Rows
Columns