Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Adding a new keyword &proc so that access to the underlying function proc is always available #231

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

Conversation

brucerennie
Copy link
Collaborator

Since it is possible to shadow the Unicon function proc, I have added a new keyword &proc that returns this internal function so that no matter what the programmer may do in terms of shadowing any of the internal Unicon functions, there is always a way to access the proc function which then allows access to all other internal Unicon functions.

@brucerennie brucerennie marked this pull request as draft April 20, 2021 12:31
@Jafaral Jafaral changed the title Adding a new keyord &proc so that access to the underlying function proc is always available Adding a new keyword &proc so that access to the underlying function proc is always available Apr 22, 2021
@brucerennie brucerennie marked this pull request as ready for review April 23, 2021 06:39
@Jafaral
Copy link
Member

Jafaral commented Apr 26, 2021

I'd like &proc to be added as part of PR that introduces a few other new pending keywords, since adding a keyword requires pumping up the icode version. Might as well do it in one fell swoop!

Any thoughts @cjeffery ?

@Jafaral Jafaral marked this pull request as draft October 1, 2021 05:32
Add a new KDef(proc,K_PROC), new Keyword function proc and #define for new keyword.

Update unigram.icn to register new keyword proc.

Signed-off-by: Bruce Rennie <[email protected]>
@brucerennie brucerennie marked this pull request as ready for review June 2, 2023 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants