fix some things in debug.rkt #3

Merged
AlexKnauth merged 1 commits from fix into master 10 years ago
AlexKnauth commented 10 years ago (Migrated from github.com)

This fixes the problem that report was evaluating the expression twice, meaning

(report (displayln "hi"))

Shows the output:

hi
(displayln hi) = #<void>
hi

This pull request fixes that and makes sure that the expression is evaluated only once.
report-apply also had that problem.

I also simplified the implementations of report*, time-repeat*, and compare, and made them more hygienic, so that they do not throw away so much lexical context and source location info.

This fixes the problem that `report` was evaluating the expression twice, meaning ``` racket (report (displayln "hi")) ``` Shows the output: ``` hi (displayln hi) = #<void> hi ``` This pull request fixes that and makes sure that the expression is evaluated only once. `report-apply` also had that problem. I also simplified the implementations of `report*`, `time-repeat*`, and `compare`, and made them more hygienic, so that they do not throw away so much lexical context and source location info.
mbutterick commented 10 years ago (Migrated from github.com)

Thanks. I didn’t know one could spit the ellipsis from the pattern variable. Good to know

Thanks. I didn’t know one could spit the ellipsis from the pattern variable. Good to know
mbutterick commented 10 years ago (Migrated from github.com)

Spit → split ;)

Spit → split ;)
The pull request has been merged as 943d422d5f.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b fix master
git pull origin fix

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff fix
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No project
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: mbutterick/sugar#3
Loading…
There is no content yet.