diff --git a/actix-router/CHANGES.md b/actix-router/CHANGES.md
index ee6d1aac..40dcf7e2 100644
--- a/actix-router/CHANGES.md
+++ b/actix-router/CHANGES.md
@@ -4,7 +4,7 @@
 * Fix segment interpolation leaving `Path` in unintended state after matching. [#368]
 * Path tail pattern now works as expected after a dynamic segment (e.g. `/user/{uid}/*`). [#366]
 * Fixed a bug where, in multi-patterns, static patterns are interpreted as regex. [#366]
-* Rename `Path::{len => segment_count}` to be more accurate. [#370]
+* Rename `Path::{len => segment_count}` to be more descriptive of it's purpose. [#370]
 
 [#368]: https://github.com/actix/actix-net/pull/368
 [#366]: https://github.com/actix/actix-net/pull/366
diff --git a/actix-router/src/resource.rs b/actix-router/src/resource.rs
index 13d7482c..e8556809 100644
--- a/actix-router/src/resource.rs
+++ b/actix-router/src/resource.rs
@@ -25,17 +25,19 @@ const REGEX_FLAGS: &str = "(?s-m)";
 pub struct ResourceDef {
     id: u16,
 
-    /// Stores
+    /// Pattern type.
     pat_type: PatternType,
 
-    /// Optional name of resource definition.
+    /// Optional name of resource definition. Defaults to "".
     name: String,
 
     /// Pattern that generated the resource definition.
-    // TODO: Sort of, in dynamic set pattern type it is blank, consider change.
+    // TODO: Sort of, in dynamic set pattern type it is blank, consider change to option.
     pattern: String,
 
+    /// List of elements that compose the pattern, in order.
     ///
+    /// `None` with pattern type is DynamicSet.
     elements: Option<Vec<PatternElement>>,
 }