why `pub` when you can `pub(crate)`?

1 files changed, 7 insertions, 7 deletions

diff --git a/src/makefile/target.rs b/src/makefile/target.rs
index 1fbc75b..443eee5 100644
--- a/src/makefile/target.rs
+++ b/src/makefile/target.rs
@@ -8,11 +8,11 @@ use crate::makefile::command_line::CommandLine;
 use super::Makefile;
 
 #[derive(PartialEq, Eq, Clone, Debug)]
-pub struct Target {
-    pub name: String,
-    pub prerequisites: Vec<String>,
-    pub commands: Vec<CommandLine>,
-    pub already_updated: Cell<bool>,
+pub(crate) struct Target {
+    pub(crate) name: String,
+    pub(crate) prerequisites: Vec<String>,
+    pub(crate) commands: Vec<CommandLine>,
+    pub(crate) already_updated: Cell<bool>,
 }
 
 impl Target {
@@ -22,7 +22,7 @@ impl Target {
             .ok()
     }
 
-    pub fn newer_than(&self, other: &Target) -> Option<bool> {
+    pub(crate) fn newer_than(&self, other: &Target) -> Option<bool> {
         let self_updated = self.already_updated.get();
         let other_updated = other.already_updated.get();
         Some(match (self.modified_time(), other.modified_time()) {
@@ -48,7 +48,7 @@ impl Target {
         exists && newer_than_all_dependencies
     }
 
-    pub fn update(&self, file: &Makefile) {
+    pub(crate) fn update(&self, file: &Makefile) {
         for prereq in &self.prerequisites {
             file.update_target(prereq);
         }