Make some comments clearer. (#665)

teymour-aldridge · web-flow · commit 31d08888e4a8 · 2020-05-23T20:29:58.000-10:00
diff --git a/juniper/src/types/scalars.rs b/juniper/src/types/scalars.rs
@@ -261,7 +261,7 @@ where
     }
 
     fn from_str<'a>(value: ScalarToken<'a>) -> ParseScalarResult<'a, S> {
-        // Bools are parsed on it's own. This should not hit this code path
+        // Bools are parsed separately - they shouldn't reach this code path
         Err(ParseError::UnexpectedToken(Token::Scalar(value)))
     }
 }
@@ -338,7 +338,7 @@ impl<T> EmptyMutation<T> {
     }
 }
 
-// This is safe due to never using `T`.
+// This is safe because `T` is never used.
 unsafe impl<T> Send for EmptyMutation<T> {}
 
 impl<S, T> GraphQLType<S> for EmptyMutation<T>

Original file line number	Diff line number	Diff line change
`@@ -261,7 +261,7 @@ where`
`261`	`261`	`}`
`262`	`262`
`263`	`263`	`fn from_str<'a>(value: ScalarToken<'a>) -> ParseScalarResult<'a, S> {`
`264`		`- // Bools are parsed on it's own. This should not hit this code path`
	`264`	`+ // Bools are parsed separately - they shouldn't reach this code path`
`265`	`265`	`Err(ParseError::UnexpectedToken(Token::Scalar(value)))`
`266`	`266`	`}`
`267`	`267`	`}`
`@@ -338,7 +338,7 @@ impl<T> EmptyMutation<T> {`
`338`	`338`	`}`
`339`	`339`	`}`
`340`	`340`
`341`		-// This is safe due to never using `T`.
	`341`	+// This is safe because `T` is never used.
`342`	`342`	`unsafe impl<T> Send for EmptyMutation<T> {}`
`343`	`343`
`344`	`344`	`impl<S, T> GraphQLType<S> for EmptyMutation<T>`