nginxinc / ngx-rust

Rust binding for NGINX
Apache License 2.0
713 stars 59 forks source link

fix: use std::ffi::c_char instead of platform specific int #66

Closed dekobon closed 4 months ago

dekobon commented 4 months ago

Proposed changes

This change reverts a previous change that did platform detection and set a type correspondingly to u8 or i8. By using std::ffi::c_char we get a portable way to do the same thing.

This PR fixes a shortcoming with the change in commit: d66f3418d64a4acf18d6b9139db89cd0ec775690

Checklist

Before creating a PR, run through this checklist and mark each as complete.

dekobon commented 4 months ago

@bavshin-f5 Thank you for the correction regarding the type usage.