scott/rudus
1
0
Fork 0
Code Issues 12 Pull Requests Actions Packages Projects Releases Wiki Activity

Panics should have nice information #6

New Issue
Closed
opened 2024-12-05 17:03:22 +00:00 by scott · 3 comments
scott commented 2024-12-05 17:03:22 +00:00
Owner
Copy Link

Right now, they just cause Rust to panic. We can do better just by printing the error message. But we should get to good panic information showing the offending line and the message, and with no matches, the scrutinee and the patterns at issue.

Right now, they just cause Rust to panic. We can do better just by printing the error message. But we should get to good panic information showing the offending line and the message, and with `no match`es, the scrutinee and the patterns at issue.
scott added the
errors
core
 labels 2024-12-05 17:03:22 +00:00
scott commented 2024-12-08 06:42:45 +00:00
Author
Owner
Copy Link

Canvas the Janet codebase to see where panics happen, and make sure we have at-least-as-good error messages in every situation.

Canvas the Janet codebase to see where panics happen, and make sure we have at-least-as-good error messages in every situation.
scott added this to the Parity or improvement over Janet-based interpreter milestone 2024-12-11 05:36:39 +00:00
scott modified the milestone from Parity or improvement over Janet-based interpreter to Minimal replacement for Janet-based Ludus 2024-12-12 00:19:28 +00:00
scott commented 2024-12-12 00:20:15 +00:00
Author
Owner
Copy Link

Moving to minimal replacement, because, uh, I need to debug Prelude, and having at all reasonable errors including locations is deeply necessary for debugging Ludus code.

Moving to minimal replacement, because, uh, I need to debug Prelude, and having at all reasonable errors including locations is deeply necessary for debugging Ludus code.
scott commented 2024-12-12 06:22:15 +00:00
Author
Owner
Copy Link

Done, with 5e73c5cb3b

Done, with https://alea.ludus.dev/scott/rudus/commit/5e73c5cb3b37ce31f8306d9e169cd181eaf194b8
scott closed this issue 2024-12-12 06:22:15 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
bug

   
bytecode

   
core

   
errors

   
feature

   
optimization
No Label
bug
bytecode
core
errors
feature
optimization
Milestone
Clear milestone
No items
No Milestone
Minimal replacement for Janet-based Ludus
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
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: scott/rudus#6
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?