You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2022/07/16 00:02:09 UTC

[GitHub] [commons-io] garydgregory commented on a diff in pull request #367: (chore) Fix minor typos

garydgregory commented on code in PR #367:
URL: https://github.com/apache/commons-io/pull/367#discussion_r922582525


##########
src/main/java/org/apache/commons/io/StandardLineSeparator.java:
##########
@@ -28,7 +28,7 @@
 public enum StandardLineSeparator {
 
     /**
-     * Carriage return. This is the line ending used on Macos 9 and earlier.
+     * Carriage return. This is the line ending used on Mac OS 9 and earlier.

Review Comment:
   The proper spelling per Apple is "macOS"



##########
src/main/java/org/apache/commons/io/input/ReversedLinesFileReader.java:
##########
@@ -404,7 +404,7 @@ public String readLine() throws IOException {
         while (line == null) {
             currentFilePart = currentFilePart.rollOver();
             if (currentFilePart == null) {
-                // no more fileparts: we're done, leave line set to null
+                // no more fileParts: we're done, leave line set to null

Review Comment:
   This is still wrong: The type name is `FilePart`.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org