summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBryant Mairs <bryant@mai.rs>2019-01-23 07:18:32 -0800
committerBryant Mairs <bryant@mai.rs>2019-02-02 16:32:00 -0800
commitfa9cb78b4a04e6cbb12c3edc3efb9486d0f13b37 (patch)
tree0e49f06cd7cc6bd0ea9084e9e8af3040b0012bba
parentcd1e16d1afb70ed510565c52d053b40a9f7c6975 (diff)
downloadrust-libc-fa9cb78b4a04e6cbb12c3edc3efb9486d0f13b37.tar.gz
Check for Debug impls for all types
This was not compile-tested on all platforms, but instead all `pub enum` types had a `Debug` impl derived for them.
-rw-r--r--src/cloudabi/mod.rs2
-rw-r--r--src/fuchsia/mod.rs6
-rw-r--r--src/lib.rs2
-rw-r--r--src/redox/mod.rs2
-rw-r--r--src/unix/bsd/apple/mod.rs1
-rw-r--r--src/unix/bsd/freebsdlike/dragonfly/mod.rs1
-rw-r--r--src/unix/bsd/freebsdlike/mod.rs1
-rw-r--r--src/unix/bsd/netbsdlike/mod.rs2
-rw-r--r--src/unix/haiku/mod.rs1
-rw-r--r--src/unix/mod.rs4
-rw-r--r--src/unix/notbsd/android/mod.rs8
-rw-r--r--src/unix/notbsd/emscripten.rs1
-rw-r--r--src/unix/notbsd/linux/mod.rs1
-rw-r--r--src/unix/notbsd/mod.rs1
-rw-r--r--src/unix/solaris/mod.rs1
-rw-r--r--src/unix/uclibc/mod.rs2
-rw-r--r--src/windows/mod.rs5
17 files changed, 36 insertions, 5 deletions
diff --git a/src/cloudabi/mod.rs b/src/cloudabi/mod.rs
index df11de002e..520ed8deee 100644
--- a/src/cloudabi/mod.rs
+++ b/src/cloudabi/mod.rs
@@ -122,7 +122,9 @@ pub const PTHREAD_STACK_MIN: ::size_t = 1024;
pub const SOCK_DGRAM: ::c_int = 128;
pub const SOCK_STREAM: ::c_int = 130;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum FILE {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos_t {} // TODO: fill this out with a struct
extern {
diff --git a/src/fuchsia/mod.rs b/src/fuchsia/mod.rs
index e785fabf8e..351ec7e37d 100644
--- a/src/fuchsia/mod.rs
+++ b/src/fuchsia/mod.rs
@@ -100,9 +100,13 @@ pub type c_ulong = u64;
// FIXME: why are these uninhabited types? that seems... wrong?
// Presumably these should be `()` or an `extern type` (when that stabilizes).
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum DIR {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum locale_t {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos64_t {} // TODO: fill this out with a struct
// PUB_STRUCT
@@ -3042,7 +3046,9 @@ f! {
#[link(name = "fdio")]
extern {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum FILE {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos_t {} // TODO: fill this out with a struct
extern {
diff --git a/src/lib.rs b/src/lib.rs
index 03e78041ab..86ab9b1368 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -156,6 +156,8 @@
#![cfg_attr(feature = "rustc-dep-of-std", no_core)]
#![cfg_attr(feature = "rustc-dep-of-std", allow(warnings))]
#![cfg_attr(not(any(feature = "use_std", feature = "rustc-dep-of-std")), no_std)]
+// Enable lints
+#![cfg_attr(feature = "extra_traits", deny(missing_debug_implementations))]
#[cfg(all(not(cross_platform_docs), feature = "use_std"))]
extern crate std as core;
diff --git a/src/redox/mod.rs b/src/redox/mod.rs
index 8dabd9063e..cfe7fd613b 100644
--- a/src/redox/mod.rs
+++ b/src/redox/mod.rs
@@ -219,7 +219,9 @@ pub const SIGIO: ::c_int = 29;
pub const SIGPWR: ::c_int = 30;
pub const SIGSYS: ::c_int = 31;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum FILE {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos_t {} // TODO: fill this out with a struct
// intentionally not public, only used for fd_set
diff --git a/src/unix/bsd/apple/mod.rs b/src/unix/bsd/apple/mod.rs
index 08995917d4..bb48d22709 100644
--- a/src/unix/bsd/apple/mod.rs
+++ b/src/unix/bsd/apple/mod.rs
@@ -34,6 +34,7 @@ pub type posix_spawn_file_actions_t = *mut ::c_void;
pub type key_t = ::c_int;
pub type shmatt_t = ::c_ushort;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
s! {
diff --git a/src/unix/bsd/freebsdlike/dragonfly/mod.rs b/src/unix/bsd/freebsdlike/dragonfly/mod.rs
index 992d0082e4..75afeb28fd 100644
--- a/src/unix/bsd/freebsdlike/dragonfly/mod.rs
+++ b/src/unix/bsd/freebsdlike/dragonfly/mod.rs
@@ -17,6 +17,7 @@ pub type fsfilcnt_t = u64;
pub type sem_t = *mut sem;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum sem {}
s! {
diff --git a/src/unix/bsd/freebsdlike/mod.rs b/src/unix/bsd/freebsdlike/mod.rs
index 22c11b3698..a402269a47 100644
--- a/src/unix/bsd/freebsdlike/mod.rs
+++ b/src/unix/bsd/freebsdlike/mod.rs
@@ -16,6 +16,7 @@ pub type speed_t = ::c_uint;
pub type nl_item = ::c_int;
pub type id_t = i64;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
s! {
diff --git a/src/unix/bsd/netbsdlike/mod.rs b/src/unix/bsd/netbsdlike/mod.rs
index 291c081b97..dfe2051836 100644
--- a/src/unix/bsd/netbsdlike/mod.rs
+++ b/src/unix/bsd/netbsdlike/mod.rs
@@ -13,7 +13,9 @@ pub type clockid_t = ::c_int;
pub type id_t = ::uint32_t;
pub type sem_t = *mut sem;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum sem {}
s! {
diff --git a/src/unix/haiku/mod.rs b/src/unix/haiku/mod.rs
index 2aa5d13a67..a7fdffdc42 100644
--- a/src/unix/haiku/mod.rs
+++ b/src/unix/haiku/mod.rs
@@ -31,6 +31,7 @@ pub type nl_item = ::c_int;
pub type id_t = i32;
pub type idtype_t = ::c_uint;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
s! {
diff --git a/src/unix/mod.rs b/src/unix/mod.rs
index 88e13c58c7..bcd16f83ef 100644
--- a/src/unix/mod.rs
+++ b/src/unix/mod.rs
@@ -41,7 +41,9 @@ pub type in_port_t = u16;
pub type sighandler_t = ::size_t;
pub type cc_t = ::c_uchar;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum DIR {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum locale_t {}
s! {
@@ -365,7 +367,9 @@ cfg_if! {
}
}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum FILE {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos_t {} // TODO: fill this out with a struct
extern {
diff --git a/src/unix/notbsd/android/mod.rs b/src/unix/notbsd/android/mod.rs
index de4ff593a0..0e2eebf056 100644
--- a/src/unix/notbsd/android/mod.rs
+++ b/src/unix/notbsd/android/mod.rs
@@ -264,7 +264,7 @@ impl std::fmt::Debug for dirent {
.field("d_off", &self.d_off)
.field("d_reclen", &self.d_reclen)
.field("d_type", &self.d_type)
- .field("d_name", &self.d_name)
+ // FIXME: .field("d_name", &self.d_name)
.finish()
}
}
@@ -302,7 +302,7 @@ impl std::fmt::Debug for dirent64 {
.field("d_off", &self.d_off)
.field("d_reclen", &self.d_reclen)
.field("d_type", &self.d_type)
- .field("d_name", &self.d_name)
+ // FIXME: .field("d_name", &self.d_name)
.finish()
}
}
@@ -374,7 +374,7 @@ impl std::fmt::Debug for lastlog {
f.debug_struct("lastlog")
.field("ll_time", &self.ll_time)
.field("ll_line", &self.ll_line)
- .field("ll_host", &self.ll_host)
+ // FIXME: .field("ll_host", &self.ll_host)
.finish()
}
}
@@ -425,7 +425,7 @@ impl std::fmt::Debug for utmp {
.field("ut_line", &self.ut_line)
.field("ut_id", &self.ut_id)
.field("ut_user", &self.ut_user)
- .field("ut_host", &self.ut_host)
+ // FIXME: .field("ut_host", &self.ut_host)
.field("ut_exit", &self.ut_exit)
.field("ut_session", &self.ut_session)
.field("ut_tv", &self.ut_tv)
diff --git a/src/unix/notbsd/emscripten.rs b/src/unix/notbsd/emscripten.rs
index 28791d8fd5..069631c1ff 100644
--- a/src/unix/notbsd/emscripten.rs
+++ b/src/unix/notbsd/emscripten.rs
@@ -35,6 +35,7 @@ pub type c_long = i32;
pub type c_ulong = u32;
pub type nlink_t = u32;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos64_t {} // TODO: fill this out with a struct
s! {
diff --git a/src/unix/notbsd/linux/mod.rs b/src/unix/notbsd/linux/mod.rs
index 31fc37922a..3150887ab0 100644
--- a/src/unix/notbsd/linux/mod.rs
+++ b/src/unix/notbsd/linux/mod.rs
@@ -38,6 +38,7 @@ pub type Elf64_Sxword = i64;
pub type Elf32_Section = u16;
pub type Elf64_Section = u16;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos64_t {} // TODO: fill this out with a struct
s! {
diff --git a/src/unix/notbsd/mod.rs b/src/unix/notbsd/mod.rs
index 4b06e6e6f6..7913508455 100644
--- a/src/unix/notbsd/mod.rs
+++ b/src/unix/notbsd/mod.rs
@@ -8,6 +8,7 @@ pub type clockid_t = ::c_int;
pub type key_t = ::c_int;
pub type id_t = ::c_uint;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
s! {
diff --git a/src/unix/solaris/mod.rs b/src/unix/solaris/mod.rs
index c991e62045..c10f4c1844 100644
--- a/src/unix/solaris/mod.rs
+++ b/src/unix/solaris/mod.rs
@@ -36,6 +36,7 @@ pub type nl_item = ::c_int;
pub type id_t = ::c_int;
pub type idtype_t = ::c_uint;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
s! {
diff --git a/src/unix/uclibc/mod.rs b/src/unix/uclibc/mod.rs
index c04c22aad9..e0620048dc 100644
--- a/src/unix/uclibc/mod.rs
+++ b/src/unix/uclibc/mod.rs
@@ -24,8 +24,10 @@ pub type nfds_t = ::c_ulong;
pub type nl_item = ::c_int;
pub type idtype_t = ::c_uint;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos64_t {} // TODO: fill this out with a struct
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
s! {
diff --git a/src/windows/mod.rs b/src/windows/mod.rs
index 4bea45980b..9deaf9b995 100644
--- a/src/windows/mod.rs
+++ b/src/windows/mod.rs
@@ -47,6 +47,7 @@ cfg_if! {
pub type off_t = i32;
pub type dev_t = u32;
pub type ino_t = u16;
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum timezone {}
pub type time64_t = i64;
@@ -201,7 +202,9 @@ pub const SIG_ERR: ::c_int = -1;
#[link(name = "libcmt", cfg(target_feature = "crt-static"))]
extern {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum FILE {}
+#[cfg_attr(feature = "extra_traits", derive(Debug))]
pub enum fpos_t {} // TODO: fill this out with a struct
extern {
@@ -446,4 +449,4 @@ cfg_if! {
} else {
// Unknown target_env
}
-} \ No newline at end of file
+}