From f9fdd60e2ebbaef94c5e297bebd5b5345957d612 Mon Sep 17 00:00:00 2001 From: Andrew Gallant Date: Mon, 1 Jul 2019 10:18:57 -0400 Subject: [PATCH] Update docs for serde_if_integer128 The docs imply that the only consideration for using the serde_if_integer128 macro is support for older versions of Rust, but Serde's build configuration for 128-bit integers is also gated on the target platform. For example, if Serde is being compiled for an emscripten target, then it will not provide 128-bit integer APIs. See also: https://github.com/BurntSushi/rust-csv/issues/158 --- serde/src/integer128.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/serde/src/integer128.rs b/serde/src/integer128.rs index 7a24e50a..01d6f34a 100644 --- a/serde/src/integer128.rs +++ b/serde/src/integer128.rs @@ -1,13 +1,13 @@ /// Conditional compilation depending on whether Serde is built with support for /// 128-bit integers. /// -/// Data formats that wish to support Rust compiler versions older than 1.26 may -/// place the i128 / u128 methods of their Serializer and Deserializer behind -/// this macro. +/// Data formats that wish to support Rust compiler versions older than 1.26 +/// (or targets that lack 128-bit integers) may place the i128 / u128 methods +/// of their Serializer and Deserializer behind this macro. /// -/// Data formats that require a minimum Rust compiler version of at least 1.26 -/// do not need to bother with this macro and may assume support for 128-bit -/// integers. +/// Data formats that require a minimum Rust compiler version of at least 1.26, +/// or do not target platforms that lack 128-bit integers, do not need to +/// bother with this macro and may assume support for 128-bit integers. /// /// ```edition2018 /// # use serde::private::ser::Error;