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

runtime: Use proper usize type for lengths #27

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

hardliner66
Copy link

This PR changes the used integer types to usize for cases where it should be platform dependent, in order to compile zigself for other platforms like armv7.

Depends on #25.

@hardliner66 hardliner66 force-pushed the fix_code_for_armv7 branch 4 times, most recently from 6d2456f to 1f10cd3 Compare August 9, 2023 12:19
@sin-ack
Copy link
Owner

sin-ack commented Aug 9, 2023

If you rebase this on master, it should show the relevant change only. :)

@sin-ack sin-ack linked an issue Aug 9, 2023 that may be closed by this pull request
@hardliner66 hardliner66 changed the title Draft: Use proper usize type for lengths Use proper usize type for lengths Aug 9, 2023
@hardliner66
Copy link
Author

Rebased onto master.

Copy link
Owner

@sin-ack sin-ack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in commit message: "lenghts"

src/runtime/primitives/system_call.zig Outdated Show resolved Hide resolved
@hardliner66
Copy link
Author

I also updated the vendored dependency. That should have already been part of the 0.11.0 update, but it kinda fell through.

@hardliner66 hardliner66 requested a review from sin-ack August 9, 2023 21:12
@sin-ack sin-ack self-assigned this Aug 13, 2023
@sin-ack
Copy link
Owner

sin-ack commented Sep 6, 2023

Sorry for taking so long to look at a trivial patch. I'll give this a lookover tomorrow, I promise. 🙇‍♂️

@hardliner66
Copy link
Author

Sorry for taking so long to look at a trivial patch. I'll give this a lookover tomorrow, I promise. 🙇‍♂️

Don't worry about it. :)

Copy link
Owner

@sin-ack sin-ack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I broke my promise :^( Just a couple quick nitpicks, and this is good to go.

src/runtime/objects/activation.zig Show resolved Hide resolved
src/runtime/primitives/system_call.zig Outdated Show resolved Hide resolved
src/runtime/primitives/system_call.zig Outdated Show resolved Hide resolved
@hardliner66
Copy link
Author

I fixed the error texts and rebased onto master.

Copy link
Owner

@sin-ack sin-ack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please also squash the commits into a single one? The PR title seems like a good title for the commit, please include the category as well.

@hardliner66 hardliner66 force-pushed the fix_code_for_armv7 branch 2 times, most recently from 6902558 to 4af0346 Compare September 28, 2023 19:03
@hardliner66 hardliner66 changed the title Use proper usize type for lengths runtime: Use proper usize type for lengths Sep 28, 2023
@hardliner66
Copy link
Author

@sin-ack Squashed the commits into one

@sin-ack sin-ack enabled auto-merge (rebase) September 28, 2023 21:37
Copy link
Owner

@sin-ack sin-ack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks for doing this, and thanks for being patient with me. :)

@sin-ack sin-ack merged commit 291b2d4 into sin-ack:master Sep 28, 2023
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.

Doesn't cross-compile to armv7
2 participants