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

chore: add buffer_scan script #325

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from
Draft

Conversation

alandtse
Copy link
Collaborator

@alandtse alandtse commented Jul 4, 2024

python ./scripts/buffer_scan.py > Buffers.md

python ./scripts/buffer_scan.py > Buffers.md
@alandtse alandtse changed the title feat: add initial buffer_scan chore: add initial buffer_scan Jul 4, 2024
@alandtse
Copy link
Collaborator Author

alandtse commented Jul 4, 2024

@FlayaN fyi. The issue you reported is probably caused by the preprocessor including common files and messing up the line numbering. Probably could solve it by disabling preprocessor includes. I haven't had time to look at that though.

I've also considered moving this to its own repo since it's python based and needs dependencies that this repo doesn't really need to have.

@alandtse alandtse changed the title chore: add initial buffer_scan chore: add buffer_scan script Jul 4, 2024
@doodlum
Copy link
Owner

doodlum commented Aug 30, 2024

This should probably be closed, or put into a separate tools folder or repo

@alandtse
Copy link
Collaborator Author

Not sure it matters one way or the other if it's left open as draft since we can't accidentally merge it in. It's mainly so someone can build the page if I drop off the face of the earth.

That said, I may move it to a separate repo this weekend if I have the time/motivation.

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