Merge pull request #201 from yuankunzhang/minor-improvements

code and docs minor improvements

src/items/builder.rs

@@ -11,7 +11,7 @@ use super::{date, relative, time, weekday, year};
 /// date and time using the `set_base()` method before calling `build()`, or
 /// leave it unset to use the current date and time as the base.
 #[derive(Debug, Default)]
-pub struct DateTimeBuilder {
+pub(crate) struct DateTimeBuilder {
     base: Option<DateTime<FixedOffset>>,
     timestamp: Option<f64>,
     date: Option<date::Date>,
@@ -289,7 +289,7 @@ impl DateTimeBuilder {
     }
 }
 
-#[allow(clippy::too_many_arguments)]
+#[allow(clippy::too_many_arguments, deprecated)]
 fn new_date(
     year: i32,
     month: u32,

src/items/combined.rs

@@ -21,16 +21,12 @@ use winnow::{
 
 use crate::items::space;
 
-use super::{
-    date::{self, Date},
-    primitive::s,
-    time::{self, Time},
-};
+use super::{date, primitive::s, time};
 
 #[derive(PartialEq, Debug, Clone, Default)]
 pub(crate) struct DateTime {
-    pub(crate) date: Date,
-    pub(crate) time: Time,
+    pub(crate) date: date::Date,
+    pub(crate) time: time::Time,
 }
 
 pub(crate) fn parse(input: &mut &str) -> ModalResult<DateTime> {

src/items/epoch.rs

@@ -1,6 +1,20 @@
 // For the full copyright and license information, please view the LICENSE
 // file that was distributed with this source code.
 
+//! From the GNU docs:
+//!
+//! > If you precede a number with ‘@’, it represents an internal timestamp as
+//! > a count of seconds.  The number can contain an internal decimal point
+//! > (either ‘.’ or ‘,’); any excess precision not supported by the internal
+//! > representation is truncated toward minus infinity.  Such a number cannot
+//! > be combined with any other date item, as it specifies a complete
+//! > timestamp.
+//! >
+//! >    On most hosts, these counts ignore the presence of leap seconds.  For
+//! > example, on most hosts ‘@1483228799’ represents 2016-12-31 23:59:59 UTC,
+//! > ‘@1483228800’ represents 2017-01-01 00:00:00 UTC, and there is no way to
+//! > represent the intervening leap second 2016-12-31 23:59:60 UTC.
+
 use winnow::{combinator::preceded, ModalResult, Parser};
 
 use super::primitive::{float, s};

src/items/mod.rs

@@ -21,14 +21,13 @@
 //!  - [`combined`]
 //!  - [`date`]
 //!  - [`epoch`]
+//!  - [`pure`]
 //!  - [`relative`]
 //!  - [`time`]
 //!  - [`timezone`]
 //!  - [`weekday`]
 //!  - [`year`]
 
-#![allow(deprecated)]
-
 // date and time items
 mod combined;
 mod date;