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

Add missing reference doc for nth/3 builtin. #197

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chuckhoupt
Copy link
Member

Fixes #196

@chuckhoupt chuckhoupt requested a review from kenbowen August 22, 2021 12:42
@chuckhoupt
Copy link
Member Author

You can see built branch at: https://nth-ref-doc--alsprolog.netlify.app/docs/ref/nth.html

@chuckhoupt chuckhoupt added the bug label Aug 22, 2021
@kenbowen
Copy link
Member

The nth/3 doc is good to go, despite the following:
It appears that the dfn of nth/3 is currently contained in builtins/simplio.pro, but it was originally in library/listutl2.pro,
as seen by some commented-out exports, along with a companion position/3,4.
If they're going to be in a builtins/*.pro file, both nth/3 and position/3,4 really ought to be in blt_term.pro.
I'll open a low-priority issue to move them. I suspect that neither of us did any "doc processing" on simplio.pro, because the companion position/3,4 are undoc'd like nth/3 was.
I'll open another issue to doc-process simple.pro.

@kenbowen kenbowen closed this Aug 22, 2021
@chuckhoupt chuckhoupt reopened this Aug 23, 2021
@chuckhoupt
Copy link
Member Author

[re-opened pull after accidental close]

Yea, nth/3 was moved from the libraries to builtins sometime in `96 -- I'm guessing because it was used to define other builtins (i.e. to avoid back-dependencies).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reference doc is missing for nth/3
2 participants