Skip to content

Commit 0a02b94

Browse files
taiki-etgross35
taiki-e
authored and
tgross35
committed
Define c_char at top-level and remove per-target c_char definitions
1 parent fd5b78c commit 0a02b94

File tree

102 files changed

+30
-173
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

102 files changed

+30
-173
lines changed

‎libc-test/build.rs‎

Lines changed: 1 addition & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -353,9 +353,6 @@ fn test_apple(target: &str) {
353353
// FIXME: "'__uint128' undeclared" in C
354354
"__uint128" => true,
355355

356-
// `c_char_def` is always public but not always reexported.
357-
"c_char_def" => true,
358-
359356
_ => false,
360357
}
361358
});
@@ -722,8 +719,6 @@ fn test_windows(target: &str) {
722719
"ssize_t" if !gnu => true,
723720
// FIXME: The size and alignment of this type are incorrect
724721
"time_t" if gnu && i686 => true,
725-
// `c_char_def` is always public but not always reexported.
726-
"c_char_def" => true,
727722
_ => false,
728723
});
729724

@@ -934,8 +929,6 @@ fn test_solarish(target: &str) {
934929

935930
cfg.skip_type(move |ty| match ty {
936931
"sighandler_t" => true,
937-
// `c_char_def` is always public but not always reexported.
938-
"c_char_def" => true,
939932
_ => false,
940933
});
941934

@@ -1236,8 +1229,6 @@ fn test_netbsd(target: &str) {
12361229
match ty {
12371230
// FIXME: sighandler_t is crazy across platforms
12381231
"sighandler_t" => true,
1239-
// `c_char_def` is always public but not always reexported.
1240-
"c_char_def" => true,
12411232
_ => false,
12421233
}
12431234
});
@@ -1455,8 +1446,6 @@ fn test_dragonflybsd(target: &str) {
14551446
match ty {
14561447
// sighandler_t is crazy across platforms
14571448
"sighandler_t" => true,
1458-
// `c_char_def` is always public but not always reexported.
1459-
"c_char_def" => true,
14601449
_ => false,
14611450
}
14621451
});
@@ -1615,8 +1604,6 @@ fn test_wasi(target: &str) {
16151604
}
16161605
});
16171606

1618-
cfg.skip_type(|ty| ty == "c_char_def");
1619-
16201607
// These have a different and internal type in header files and are only
16211608
// used here to generate a pointer to them in bindings so skip these tests.
16221609
cfg.skip_static(|c| c.starts_with("_CLOCK_"));
@@ -1865,9 +1852,6 @@ fn test_android(target: &str) {
18651852
// FIXME: "'__uint128' undeclared" in C
18661853
"__uint128" => true,
18671854

1868-
// `c_char_def` is always public but not always reexported.
1869-
"c_char_def" => true,
1870-
18711855
_ => false,
18721856
}
18731857
});
@@ -2619,9 +2603,6 @@ fn test_freebsd(target: &str) {
26192603
// `eventfd(2)` and things come with it are added in FreeBSD 13
26202604
"eventfd_t" if Some(13) > freebsd_ver => true,
26212605

2622-
// `c_char_def` is always public but not always reexported.
2623-
"c_char_def" => true,
2624-
26252606
_ => false,
26262607
}
26272608
});
@@ -2938,9 +2919,6 @@ fn test_emscripten(target: &str) {
29382919
// https://github.com/emscripten-core/emscripten/issues/5033
29392920
ty if ty.starts_with("epoll") => true,
29402921

2941-
// `c_char_def` is always public but not always reexported.
2942-
"c_char_def" => true,
2943-
29442922
// LFS64 types have been removed in Emscripten 3.1.44
29452923
// https://github.com/emscripten-core/emscripten/pull/19812
29462924
t => t.ends_with("64") || t.ends_with("64_t"),
@@ -3217,9 +3195,6 @@ fn test_neutrino(target: &str) {
32173195
// Does not exist in Neutrino
32183196
"locale_t" => true,
32193197

3220-
// `c_char_def` is always public but not always reexported.
3221-
"c_char_def" => true,
3222-
32233198
_ => false,
32243199
}
32253200
});
@@ -3383,8 +3358,6 @@ fn test_vxworks(target: &str) {
33833358
// FIXME
33843359
cfg.skip_type(move |ty| match ty {
33853360
"stat64" | "sighandler_t" | "off64_t" => true,
3386-
// `c_char_def` is always public but not always reexported.
3387-
"c_char_def" => true,
33883361
_ => false,
33893362
});
33903363

@@ -3732,9 +3705,6 @@ fn test_linux(target: &str) {
37323705
// FIXME: "'__uint128' undeclared" in C
37333706
"__uint128" => true,
37343707

3735-
// `c_char_def` is always public but not always reexported.
3736-
"c_char_def" => true,
3737-
37383708
t => {
37393709
if musl {
37403710
// LFS64 types have been removed in musl 1.2.4+
@@ -3964,7 +3934,7 @@ fn test_linux(target: &str) {
39643934
}
39653935
// FIXME: Requires >= 5.4 kernel headers
39663936
if name == "PTP_CLOCK_GETCAPS2"
3967-
|| name == "PTP_ENABLE_PPS2"
3937+
|| name == "PTP_ENABLE_PPS2"
39683938
|| name == "PTP_EXTTS_REQUEST2"
39693939
|| name == "PTP_PEROUT_REQUEST2"
39703940
|| name == "PTP_PIN_GETFUNC2"
@@ -4683,8 +4653,6 @@ fn test_linux_like_apis(target: &str) {
46834653
})
46844654
.skip_type(move |ty| match ty {
46854655
"Elf64_Phdr" | "Elf32_Phdr" => false,
4686-
// `c_char_def` is always public but not always reexported.
4687-
"c_char_def" => true,
46884656
_ => true,
46894657
});
46904658
cfg.generate(src_hotfix_dir().join("lib.rs"), "linux_elf.rs");
@@ -4920,8 +4888,6 @@ fn test_haiku(target: &str) {
49204888
"pthread_condattr_t" => true,
49214889
"pthread_mutexattr_t" => true,
49224890
"pthread_rwlockattr_t" => true,
4923-
// `c_char_def` is always public but not always reexported.
4924-
"c_char_def" => true,
49254891
_ => false,
49264892
}
49274893
});

‎src/fuchsia/aarch64.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
use crate::off_t;
22
use crate::prelude::*;
33

4-
pub type c_char = u8;
54
pub type __u64 = c_ulonglong;
65
pub type wchar_t = u32;
76
pub type nlink_t = c_ulong;

‎src/fuchsia/riscv64.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ use crate::off_t;
22
use crate::prelude::*;
33

44
// From psABI Calling Convention for RV64
5-
pub type c_char = u8;
65
pub type __u64 = c_ulonglong;
76
pub type wchar_t = i32;
87

‎src/fuchsia/x86_64.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
use crate::off_t;
22
use crate::prelude::*;
33

4-
pub type c_char = i8;
54
pub type wchar_t = i32;
65
pub type nlink_t = u64;
76
pub type blksize_t = c_long;

‎src/hermit.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
//! Hermit C type definitions
22
3-
pub use crate::arch::c_char_def as c_char;
43
use crate::prelude::*;
54

65
pub type c_schar = i8;

‎src/lib.rs‎

Lines changed: 29 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -38,41 +38,35 @@ cfg_if! {
3838

3939
pub use core::ffi::c_void;
4040

41-
/// Type definitions that are coupled tighter to architecture than OS.
42-
mod arch {
43-
cfg_if! {
44-
// This configuration comes from `rust-lang/rust` in `library/core/src/ffi/mod.rs`.
45-
if #[cfg(all(
46-
not(windows),
47-
// FIXME(ctest): just use `target_vendor` = "apple"` once `ctest` supports it
48-
not(any(
49-
target_os = "macos",
50-
target_os = "ios",
51-
target_os = "tvos",
52-
target_os = "watchos",
53-
target_os = "visionos",
54-
)),
55-
any(
56-
target_arch = "aarch64",
57-
target_arch = "arm",
58-
target_arch = "csky",
59-
target_arch = "hexagon",
60-
target_arch = "msp430",
61-
target_arch = "powerpc",
62-
target_arch = "powerpc64",
63-
target_arch = "riscv64",
64-
target_arch = "riscv32",
65-
target_arch = "s390x",
66-
target_arch = "xtensa",
67-
)
68-
))] {
69-
// To be reexported as `c_char`
70-
// FIXME(ctest): just name these `c_char` once `ctest` learns that these don't get
71-
// exported.
72-
pub type c_char_def = u8;
73-
} else {
74-
pub type c_char_def = i8;
75-
}
41+
cfg_if! {
42+
// This configuration comes from `rust-lang/rust` in `library/core/src/ffi/mod.rs`.
43+
if #[cfg(all(
44+
not(windows),
45+
// FIXME(ctest): just use `target_vendor` = "apple"` once `ctest` supports it
46+
not(any(
47+
target_os = "macos",
48+
target_os = "ios",
49+
target_os = "tvos",
50+
target_os = "watchos",
51+
target_os = "visionos",
52+
)),
53+
any(
54+
target_arch = "aarch64",
55+
target_arch = "arm",
56+
target_arch = "csky",
57+
target_arch = "hexagon",
58+
target_arch = "msp430",
59+
target_arch = "powerpc",
60+
target_arch = "powerpc64",
61+
target_arch = "riscv64",
62+
target_arch = "riscv32",
63+
target_arch = "s390x",
64+
target_arch = "xtensa",
65+
)
66+
))] {
67+
pub type c_char = u8;
68+
} else {
69+
pub type c_char = i8;
7670
}
7771
}
7872

‎src/sgx.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ pub type intptr_t = isize;
1919
pub type uintptr_t = usize;
2020
pub type ssize_t = isize;
2121

22-
pub type c_char = i8;
2322
pub type c_long = i64;
2423
pub type c_ulong = u64;
2524

‎src/solid/aarch64.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
pub type c_char = u8;
21
pub type wchar_t = u32;
32
pub type c_long = i64;
43
pub type c_ulong = u64;

‎src/solid/arm.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
pub type c_char = u8;
21
pub type wchar_t = u32;
32
pub type c_long = i32;
43
pub type c_ulong = u32;

‎src/switch.rs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ pub type uintptr_t = usize;
2020
pub type ssize_t = isize;
2121

2222
pub type off_t = i64;
23-
pub type c_char = u8;
2423
pub type c_long = i64;
2524
pub type c_ulong = u64;
2625
pub type wchar_t = u32;

0 commit comments

Comments
 (0)