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 %vm-config for VM introspection #560

Merged
merged 2 commits into from
Jul 20, 2024

Conversation

jpellegrini
Copy link
Contributor

Hi @egallesio !

This is one thing I am using to help debug the VM. I am reeeeeally not sure if it's worth including, but well... Here it is, if you think it'd be nice to have in STklos.

stklos> (%vm-config)
(computed-goto)

It would (when #528 is fixed) also include in the list the symbols debug-vm and stat-vm when appropriate.

@jpellegrini jpellegrini changed the title Add %vm-config for VM introspection Add %vm-config for VM introspection Jul 9, 2023
@egallesio
Copy link
Owner

Hi @jpellegrini,
I'm sorry for being so long to work on your PRs. I'm just in the process of purging most of the items on my TODO list, targeting the long awaited 2.00 version (yep!). I plan to add them just after 2.00, otherwise we will never have a stable release.

@jpellegrini
Copy link
Contributor Author

Hi @egallesio !
Of course, it's just an idea I had, and it's not that important. The new release is more important!

@egallesio
Copy link
Owner

This is not for this specific PR, but all the ones that are still open.

@egallesio egallesio merged commit f0ef5f4 into egallesio:master Jul 20, 2024
@egallesio
Copy link
Owner

I have merged your PR and modified it a bit. Instead of returning a list, %vm-config returns now a property list.

stklos> (%vm-config)
(#:computed-goto #t #:debug-vm #f #:stat-vm #t)

Thanks.

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.

None yet

2 participants