fix some things in debug.rkt #3

Merged
AlexKnauth merged 1 commits from fix into master 9 years ago
AlexKnauth commented 9 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 9 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 9 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.