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

size_t should be replaced by uint, not csize #181

Closed
stefantalpalaru opened this issue Jan 10, 2020 · 3 comments
Closed

size_t should be replaced by uint, not csize #181

stefantalpalaru opened this issue Jan 10, 2020 · 3 comments

Comments

@stefantalpalaru
Copy link

Since this tool is being used to interface with C and C++ libraries, we can't afford that whole "we'll change the C standard in our leaky abstraction" silliness, so what is an unsigned integer in C should be the same in the Nim wrapper.

Relevant Nim silliness: nim-lang/Nim#12187

@Araq
Copy link
Member

Araq commented Jan 11, 2020

Don't we have csize_t now in system.nim we can use instead?

@stefantalpalaru
Copy link
Author

Yes, csize_t will be just as good, once it reaches a release.

@Araq
Copy link
Member

Araq commented May 30, 2021

Now uses csize_t.

@Araq Araq closed this as completed May 30, 2021
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

No branches or pull requests

2 participants