Dev stacker #13

Closed
ijean wants to merge 4 commits from dev-stacker into master
ijean commented 7 years ago (Migrated from github.com)

Hello Matthew,

I've slightly changed the handle function from stacker to allow extended operations like - and / while keeping RPN notation. Simply extending the original implementation with - and / was giving incorrect results because of the way the arguments taken from the stack were used when evaluating the current operation.

Also, added an example of extending stacker with - and /. Added two extra tests.

Thanks,
Jean

Hello Matthew, I've slightly changed the handle function from stacker to allow extended operations like - and / while keeping RPN notation. Simply extending the original implementation with - and / was giving incorrect results because of the way the arguments taken from the stack were used when evaluating the current operation. Also, added an example of extending stacker with - and /. Added two extra tests. Thanks, Jean
This pull request cannot be reopened because the branch was deleted.
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 dev-stacker master
git pull origin dev-stacker

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff dev-stacker
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/beautiful-racket#13
Loading…
There is no content yet.