You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
c_void has wrong rustc support level in docs #55619
While the module c_void lays in, core::ffi, was stabilised at 1.30.0, the type c_void was stabilised at 1.1.0, which again makes the docs correct. The docs for the module core::ffi say 1.30.0.
I guess that's a matter of perspective. I think minimum versions of parent items should trickle down to child items such that you don't have to walk up the module tree in the documentation to find out whether something is available or not.
So I did the original PR to create core::ffi::c_void, and probably agree with @jethrogb in hindsight. The c_void type was originally available at std::os::raw::c_void, which should stay at 1.1.0. However, both std::ffi::c_void and core::ffi::c_void should probably be changed to 1.30.0.
Activity
Proximyst commentedon Nov 3, 2018
While the module
c_voidlays in,core::ffi, was stabilised at 1.30.0, the typec_voidwas stabilised at 1.1.0, which again makes the docs correct. The docs for the modulecore::ffisay 1.30.0.jethrogb commentedon Nov 3, 2018
I guess that's a matter of perspective. I think minimum versions of parent items should trickle down to child items such that you don't have to walk up the module tree in the documentation to find out whether something is available or not.
IsaacWoods commentedon Nov 4, 2018
So I did the original PR to create
core::ffi::c_void, and probably agree with @jethrogb in hindsight. Thec_voidtype was originally available atstd::os::raw::c_void, which should stay at1.1.0. However, bothstd::ffi::c_voidandcore::ffi::c_voidshould probably be changed to1.30.0.ExpHP commentedon Nov 6, 2018
This has happened before as well: #49651
Rollup merge of rust-lang#65930 - lzutao:new-feature-gate-c_void, r=d…