Author: rwhitcomb Date: Thu May 7 20:44:06 2020 New Revision: 1877486 URL: http://svn.apache.org/viewvc?rev=1877486&view=rev Log: PIVOT-1032: Tweak the file count report from StyleErrors a little bit.
Modified: pivot/trunk/StyleErrors.java Modified: pivot/trunk/StyleErrors.java URL: http://svn.apache.org/viewvc/pivot/trunk/StyleErrors.java?rev=1877486&r1=1877485&r2=1877486&view=diff ============================================================================== --- pivot/trunk/StyleErrors.java (original) +++ pivot/trunk/StyleErrors.java Thu May 7 20:44:06 2020 @@ -183,7 +183,7 @@ public final class StyleErrors { /** The list of file names, to be sorted by error counts. */ private static List<FileInfo> fileCountList = new ArrayList<>(); /** Number of files to list with least and most errors. */ - private static final int NUMBER_OF_FILES_TO_REPORT = 10; + private static final int NUMBER_OF_FILES_TO_REPORT = 12; /** The latest file name we're working on in the current error log. */ private static String currentFileName; /** Whether we are running on a Windows system. */ @@ -315,10 +315,10 @@ public final class StyleErrors { // The list is sorted by count, with highest count first fileCountList.sort((o1, o2) -> o2.getCount() - o1.getCount()); System.out.println(verbose ? "File error counts:" : "Files with the most errors:"); - int num = 0; + int num = 1; for (FileInfo info : fileCountList) { System.out.format(FORMAT4, info.getName(), info.getCount()); - if (!verbose && num++ > NUMBER_OF_FILES_TO_REPORT) { + if (!verbose && num++ >= NUMBER_OF_FILES_TO_REPORT) { break; } } @@ -328,13 +328,11 @@ public final class StyleErrors { // The list is sorted by count, with lowest count first fileCountList.sort((o1, o2) -> o1.getCount() - o2.getCount()); System.out.println("Files with the fewest errors:"); - num = 0; - for (FileInfo info : fileCountList) { + for (int i = NUMBER_OF_FILES_TO_REPORT; i > 0; i--) { + FileInfo info = fileCountList.get(i - 1); System.out.format(FORMAT4, info.getName(), info.getCount()); - if (num++ > NUMBER_OF_FILES_TO_REPORT) { - break; - } } + System.out.println(); } } }