Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#618 alternative to dupn #2059

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

leonid-s-usov
Copy link
Contributor

@wtlangford @nicowilliams It's old, I know, but why not?

Following my comments at #618 I am proving here my points.

The suggested FORK_1N would need to pop/push for the first time and then produce null after backtrack, so that the second branch has some input to consume. Hence a small change in the proposed naming.

Or, alternatively, we could introduce at least one instruction which produces value but doesn't consume one. This would come in handy in many places - maybe some kind of a modifier for existing opcodes. Just saying...

All tests pass.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 84.075% when pulling 145848f on leonid-s-usov:618-alternative-to-DUPN into 5b9e63e on stedolan:master.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants