Skip to content

Document initialize #680

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

Merged
merged 2 commits into from
Sep 5, 2024
Merged

Document initialize #680

merged 2 commits into from
Sep 5, 2024

Conversation

jpellegrini
Copy link
Contributor

Hi @egallesio !

I needed the CLOS initialize-instance method and couldn't find it, then I found that STklos implements it with the name "initialize"... And also found that it wasn't documented yet, so I have written some documentation. Is the explanation I gave OK?

(initialize class initargs) is documented.
@egallesio egallesio merged commit c5b63f8 into egallesio:master Sep 5, 2024
@egallesio
Copy link
Owner

This is perfect.
I have just placed the documentation with make and finally renamed initialize as initialize-instance (since its friend is called allocate-instance). Of course, initialize is still usable.

I have also added the definition of make at the end, of your explanation to explicit the relation between make, allocate-instance, and initialize-instance.

Thanks a lot for this PR.

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