Reviewers: scottb, Description: Scott,
could you review this simple patch for me? It reverts an earlier change, so that the compiler log starts counting permutations at 0 again. While less pretty, it's consistent with everything else. Thanks, kathrin Please review this at http://gwt-code-reviews.appspot.com/112803 Affected files: dev/core/src/com/google/gwt/dev/PermutationWorkerFactory.java dev/core/src/com/google/gwt/dev/jjs/JavaToJavaScriptCompiler.java Index: dev/core/src/com/google/gwt/dev/jjs/JavaToJavaScriptCompiler.java =================================================================== --- dev/core/src/com/google/gwt/dev/jjs/JavaToJavaScriptCompiler.java (revision 7071) +++ dev/core/src/com/google/gwt/dev/jjs/JavaToJavaScriptCompiler.java (working copy) @@ -216,8 +216,7 @@ PropertyOracle[] propertyOracles = permutation.getPropertyOracles(); int permutationId = permutation.getId(); Map<String, String> rebindAnswers = permutation.getRebindAnswers(); - int printId = permutationId + 1; - logger.log(TreeLogger.INFO, "Compiling permutation " + printId + "..."); + logger.log(TreeLogger.INFO, "Compiling permutation " + permutationId + "..."); long permStart = System.currentTimeMillis(); try { if (JProgram.isTracingEnabled()) { Index: dev/core/src/com/google/gwt/dev/PermutationWorkerFactory.java =================================================================== --- dev/core/src/com/google/gwt/dev/PermutationWorkerFactory.java (revision 7071) +++ dev/core/src/com/google/gwt/dev/PermutationWorkerFactory.java (working copy) @@ -249,9 +249,8 @@ List<Work> work = new ArrayList<Work>(permutations.length); for (int i = 0; i < permutations.length; ++i) { Permutation perm = permutations[i]; - int printId = perm.getId() + 1; logger.log(TreeLogger.DEBUG, - "Creating worker permutation " + printId + " of " + permutations.length); + "Creating worker permutation " + perm.getId() + " of " + permutations.length); work.add(new Work(logger, perm, resultFiles.get(i))); } -- http://groups.google.com/group/Google-Web-Toolkit-Contributors