Minor doc fixes + making directFileRead private

Author: joefarebrother

java/ql/lib/semmle/code/java/frameworks/android/ExternalStorage.qll

@@ -39,7 +39,7 @@ private predicate externalStorageFlow(DataFlow::Node node1, DataFlow::Node node2
 
 /**
  * Holds if `n` is a node that reads the contents of an external file in Android.
- * This is controlable by third-party applications, so is treated as a remote flow source.
+ * This is controllable by third-party applications, so is treated as a remote flow source.
  */
 predicate androidExternalStorageSource(DataFlow::Node n) {
   exists(DataFlow::Node externalDir, DirectFileReadExpr read |

java/ql/lib/semmle/code/java/security/FileReadWrite.qll

@@ -3,7 +3,7 @@ import java
 /**
  * Holds if `fileAccess` is directly used in the `fileReadingExpr` to read the represented file.
  */
-predicate directFileRead(Expr fileAccess, Expr fileReadingExpr) {
+private predicate directFileRead(Expr fileAccess, Expr fileReadingExpr) {
   // `fileAccess` used to construct a class that reads a file.
   exists(ClassInstanceExpr cie |
     cie = fileReadingExpr and
@@ -64,7 +64,7 @@ class DirectFileReadExpr extends Expr {
   DirectFileReadExpr() { directFileRead(_, this) }
 
   /**
-   * Gets the `Expr` representing the file that is read
+   * Gets the `Expr` representing the file that is read.
    */
   Expr getFileExpr() { directFileRead(result, this) }
 }