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

Remove libc dependency. #1117

Merged
merged 1 commit into from
Nov 24, 2022
Merged

Remove libc dependency. #1117

merged 1 commit into from
Nov 24, 2022

Conversation

hasenbanck
Copy link
Contributor

The main goal of this PR is to remove dependencies with functionality that is already present in the std or the crate.

Only size_t needs to be defined like was done with the intptr_t type before.

Also unifies the usage of std::os::raw types that where already defined in the support module and used in other parts of the crate.

@CLAassistant
Copy link

CLAassistant commented Nov 8, 2022

CLA assistant check
All committers have signed the CLA.

Use `std` functionality where possible. Only size_t needs
to be defined like was done with the `intptr_t` type before.

Also unifies the usage of `std::os::raw` types that where
already defined in the support module.
@hasenbanck
Copy link
Contributor Author

I tried to reproduce the error that the CI throws, but can't reproduce it under x86_64-pc-windows-msvc nor x86_64-unknown-linux-gnu. It also seems to come from a function that was not changed by this PR.

@bartlomieju
Copy link
Member

@piscisaureus please take a look

Copy link
Member

@littledivy littledivy left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you

@littledivy littledivy merged commit e89d532 into denoland:main Nov 24, 2022
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

4 participants