test/langtools/tools/javac/failover/CheckAttributedTree.java
author jlahoda
Wed, 29 Aug 2018 09:36:17 +0200
changeset 51563 de411d537aae
parent 49518 d0ff431a596e
permissions -rw-r--r--
8206986: Compiler support for Switch Expressions (Preview) 8207405: Compiler Tree API support for Switch Expressions (Preview) Summary: Support for switch expression, switch with rules and multiple constants for cases. Reviewed-by: jjg, mcimadamore, vromero
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     1
/*
49518
d0ff431a596e 8198314: javac hidden options violate standard syntax for options
vromero
parents: 47863
diff changeset
     2
 * Copyright (c) 2010, 2018, Oracle and/or its affiliates. All rights reserved.
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     4
 *
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     7
 * published by the Free Software Foundation.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     8
 *
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    13
 * accompanied this code).
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    14
 *
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    18
 *
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    21
 * questions.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    22
 */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    23
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    24
/*
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    25
 * @test
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    26
 * @bug 6970584 8006694 8062373 8129962
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    27
 * @summary assorted position errors in compiler syntax trees
15384
5a8d00abf076 8006694: temporarily workaround combo tests are causing time out in several platforms
vromero
parents: 15040
diff changeset
    28
 *  temporarily workaround combo tests are causing time out in several platforms
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    29
 * @library ../lib
30730
d3ce7619db2c 8076543: Add @modules as needed to the langtools tests
akulyakh
parents: 29776
diff changeset
    30
 * @modules java.desktop
35359
f04501964016 8148147: Sync up @modules from jigsaw/jake
alanb
parents: 32454
diff changeset
    31
 *          jdk.compiler/com.sun.tools.javac.api
30730
d3ce7619db2c 8076543: Add @modules as needed to the langtools tests
akulyakh
parents: 29776
diff changeset
    32
 *          jdk.compiler/com.sun.tools.javac.code
35359
f04501964016 8148147: Sync up @modules from jigsaw/jake
alanb
parents: 32454
diff changeset
    33
 *          jdk.compiler/com.sun.tools.javac.comp
f04501964016 8148147: Sync up @modules from jigsaw/jake
alanb
parents: 32454
diff changeset
    34
 *          jdk.compiler/com.sun.tools.javac.main
30730
d3ce7619db2c 8076543: Add @modules as needed to the langtools tests
akulyakh
parents: 29776
diff changeset
    35
 *          jdk.compiler/com.sun.tools.javac.tree
d3ce7619db2c 8076543: Add @modules as needed to the langtools tests
akulyakh
parents: 29776
diff changeset
    36
 *          jdk.compiler/com.sun.tools.javac.util
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    37
 * @build combo.ComboTestHelper
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    38
 * @run main CheckAttributedTree -q -r -et ERRONEOUS .
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    39
 */
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    40
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    41
import java.awt.BorderLayout;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    42
import java.awt.Color;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    43
import java.awt.Dimension;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    44
import java.awt.EventQueue;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    45
import java.awt.Font;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    46
import java.awt.GridBagConstraints;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    47
import java.awt.GridBagLayout;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    48
import java.awt.Rectangle;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    49
import java.awt.event.ActionEvent;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    50
import java.awt.event.ActionListener;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    51
import java.awt.event.MouseAdapter;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    52
import java.awt.event.MouseEvent;
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    53
import java.io.File;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    54
import java.io.IOException;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    55
import java.io.PrintStream;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    56
import java.io.PrintWriter;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    57
import java.io.StringWriter;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    58
import java.lang.reflect.Field;
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    59
import java.nio.file.FileVisitResult;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    60
import java.nio.file.Files;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    61
import java.nio.file.Path;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    62
import java.nio.file.SimpleFileVisitor;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    63
import java.nio.file.attribute.BasicFileAttributes;
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    64
import java.util.ArrayList;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    65
import java.util.Arrays;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    66
import java.util.HashSet;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    67
import java.util.List;
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    68
import java.util.Set;
26264
a09fedde76be 8044859: javac duplicates option processing when using Compiler API
jjg
parents: 15385
diff changeset
    69
import java.util.concurrent.atomic.AtomicInteger;
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
    70
import java.util.function.BiConsumer;
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    71
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    72
import javax.lang.model.element.Element;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    73
import javax.swing.DefaultComboBoxModel;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    74
import javax.swing.JComboBox;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    75
import javax.swing.JComponent;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    76
import javax.swing.JFrame;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    77
import javax.swing.JLabel;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    78
import javax.swing.JPanel;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    79
import javax.swing.JScrollPane;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    80
import javax.swing.JTextArea;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    81
import javax.swing.JTextField;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    82
import javax.swing.SwingUtilities;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    83
import javax.swing.event.CaretEvent;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    84
import javax.swing.event.CaretListener;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    85
import javax.swing.text.BadLocationException;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    86
import javax.swing.text.DefaultHighlighter;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    87
import javax.swing.text.Highlighter;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    88
import javax.tools.JavaFileObject;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    89
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    90
import com.sun.source.tree.CompilationUnitTree;
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
    91
import com.sun.source.util.TaskEvent;
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
    92
import com.sun.source.util.TaskEvent.Kind;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    93
import com.sun.source.util.TaskListener;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    94
import com.sun.tools.javac.code.Symbol;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    95
import com.sun.tools.javac.code.Type;
13077
16fb753bb5dc 7174143: encapsulate doc comment table
jjg
parents: 11055
diff changeset
    96
import com.sun.tools.javac.tree.EndPosTable;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    97
import com.sun.tools.javac.tree.JCTree;
51563
de411d537aae 8206986: Compiler support for Switch Expressions (Preview)
jlahoda
parents: 49518
diff changeset
    98
import com.sun.tools.javac.tree.JCTree.JCBreak;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
    99
import com.sun.tools.javac.tree.JCTree.JCCompilationUnit;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   100
import com.sun.tools.javac.tree.JCTree.JCImport;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   101
import com.sun.tools.javac.tree.TreeInfo;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   102
import com.sun.tools.javac.tree.TreeScanner;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   103
import com.sun.tools.javac.util.Pair;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   104
10950
e87b50888909 6921494: provide way to print javac tree tag values
jjg
parents: 7846
diff changeset
   105
import static com.sun.tools.javac.tree.JCTree.Tag.*;
e87b50888909 6921494: provide way to print javac tree tag values
jjg
parents: 7846
diff changeset
   106
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   107
import combo.ComboTestHelper;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   108
import combo.ComboInstance;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   109
import combo.ComboTestHelper.IgnoreMode;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   110
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   111
/**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   112
 * Utility and test program to check validity of tree positions for tree nodes.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   113
 * The program can be run standalone, or as a jtreg test.  In standalone mode,
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   114
 * errors can be displayed in a gui viewer. For info on command line args,
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   115
 * run program with no args.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   116
 *
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   117
 * <p>
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   118
 * jtreg: Note that by using the -r switch in the test description below, this test
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   119
 * will process all java files in the langtools/test directory, thus implicitly
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   120
 * covering any new language features that may be tested in this test suite.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   121
 */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   122
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   123
public class CheckAttributedTree {
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   124
    /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   125
     * Main entry point.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   126
     * If test.src is set, program runs in jtreg mode, and will throw an Error
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   127
     * if any errors arise, otherwise System.exit will be used, unless the gui
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   128
     * viewer is being used. In jtreg mode, the default base directory for file
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   129
     * args is the value of ${test.src}. In jtreg mode, the -r option can be
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   130
     * given to change the default base directory to the root test directory.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   131
     */
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   132
    public static void main(String... args) throws Exception {
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   133
        String testSrc = System.getProperty("test.src");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   134
        File baseDir = (testSrc == null) ? null : new File(testSrc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   135
        boolean ok = new CheckAttributedTree().run(baseDir, args);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   136
        if (!ok) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   137
            if (testSrc != null)  // jtreg mode
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   138
                throw new Error("failed");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   139
            else
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   140
                System.exit(1);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   141
        }
47863
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   142
        System.err.println("total number of compilations " + totalNumberOfCompilations);
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   143
        System.err.println("number of failed compilations " + numberOfFailedCompilations);
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   144
    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   145
47863
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   146
    static private int totalNumberOfCompilations = 0;
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   147
    static private int numberOfFailedCompilations = 0;
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   148
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   149
    /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   150
     * Run the program. A base directory can be provided for file arguments.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   151
     * In jtreg mode, the -r option can be given to change the default base
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   152
     * directory to the test root directory. For other options, see usage().
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   153
     * @param baseDir base directory for any file arguments.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   154
     * @param args command line args
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   155
     * @return true if successful or in gui mode
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   156
     */
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   157
    boolean run(File baseDir, String... args) throws Exception {
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   158
        if (args.length == 0) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   159
            usage(System.out);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   160
            return true;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   161
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   162
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   163
        List<File> files = new ArrayList<File>();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   164
        for (int i = 0; i < args.length; i++) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   165
            String arg = args[i];
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   166
            if (arg.equals("-encoding") && i + 1 < args.length)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   167
                encoding = args[++i];
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   168
            else if (arg.equals("-gui"))
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   169
                gui = true;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   170
            else if (arg.equals("-q"))
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   171
                quiet = true;
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   172
            else if (arg.equals("-v")) {
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   173
                verbose = true;
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   174
            }
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   175
            else if (arg.equals("-t") && i + 1 < args.length)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   176
                tags.add(args[++i]);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   177
            else if (arg.equals("-ef") && i + 1 < args.length)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   178
                excludeFiles.add(new File(baseDir, args[++i]));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   179
            else if (arg.equals("-et") && i + 1 < args.length)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   180
                excludeTags.add(args[++i]);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   181
            else if (arg.equals("-r")) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   182
                if (excludeFiles.size() > 0)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   183
                    throw new Error("-r must be used before -ef");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   184
                File d = baseDir;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   185
                while (!new File(d, "TEST.ROOT").exists()) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   186
                    if (d == null)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   187
                        throw new Error("cannot find TEST.ROOT");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   188
                    d = d.getParentFile();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   189
                }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   190
                baseDir = d;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   191
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   192
            else if (arg.startsWith("-"))
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   193
                throw new Error("unknown option: " + arg);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   194
            else {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   195
                while (i < args.length)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   196
                    files.add(new File(baseDir, args[i++]));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   197
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   198
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   199
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   200
        ComboTestHelper<FileChecker> cth = new ComboTestHelper<>();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   201
        cth.withIgnoreMode(IgnoreMode.IGNORE_ALL)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   202
                .withFilter(FileChecker::checkFile)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   203
                .withDimension("FILE", (x, file) -> x.file = file, getAllFiles(files))
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   204
                .run(FileChecker::new);
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   205
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   206
        if (fileCount.get() != 1)
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   207
            errWriter.println(fileCount + " files read");
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   208
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   209
        if (verbose) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   210
            System.out.println(errSWriter.toString());
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   211
        }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   212
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   213
        return (gui || !cth.info().hasFailures());
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   214
    }
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   215
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   216
    File[] getAllFiles(List<File> roots) throws IOException {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   217
        long now = System.currentTimeMillis();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   218
        ArrayList<File> buf = new ArrayList<>();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   219
        for (File file : roots) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   220
            Files.walkFileTree(file.toPath(), new SimpleFileVisitor<Path>() {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   221
                @Override
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   222
                public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   223
                    buf.add(file.toFile());
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   224
                    return FileVisitResult.CONTINUE;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   225
                }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   226
            });
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   227
        }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   228
        long delta = System.currentTimeMillis() - now;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   229
        System.err.println("All files = " + buf.size() + " " + delta);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   230
        return buf.toArray(new File[buf.size()]);
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   231
    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   232
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   233
    /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   234
     * Print command line help.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   235
     * @param out output stream
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   236
     */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   237
    void usage(PrintStream out) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   238
        out.println("Usage:");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   239
        out.println("  java CheckAttributedTree options... files...");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   240
        out.println("");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   241
        out.println("where options include:");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   242
        out.println("-q        Quiet: don't report on inapplicable files");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   243
        out.println("-gui      Display returns in a GUI viewer");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   244
        out.println("-v        Verbose: report on files as they are being read");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   245
        out.println("-t tag    Limit checks to tree nodes with this tag");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   246
        out.println("          Can be repeated if desired");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   247
        out.println("-ef file  Exclude file or directory");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   248
        out.println("-et tag   Exclude tree nodes with given tag name");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   249
        out.println("");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   250
        out.println("files may be directories or files");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   251
        out.println("directories will be scanned recursively");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   252
        out.println("non java files, or java files which cannot be parsed, will be ignored");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   253
        out.println("");
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   254
    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   255
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   256
    class FileChecker extends ComboInstance<FileChecker> {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   257
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   258
        File file;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   259
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   260
        boolean checkFile() {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   261
            if (!file.exists()) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   262
                error("File not found: " + file);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   263
                return false;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   264
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   265
            if (excludeFiles.contains(file)) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   266
                if (!quiet)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   267
                    error("File " + file + " excluded");
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   268
                return false;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   269
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   270
            if (!file.getName().endsWith(".java")) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   271
                if (!quiet)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   272
                    error("File " + file + " ignored");
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   273
                return false;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   274
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   275
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   276
            return true;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   277
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   278
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   279
        public void doWork() {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   280
            if (!file.exists()) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   281
                error("File not found: " + file);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   282
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   283
            if (excludeFiles.contains(file)) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   284
                if (!quiet)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   285
                    error("File " + file + " excluded");
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   286
                return;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   287
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   288
            if (!file.getName().endsWith(".java")) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   289
                if (!quiet)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   290
                    error("File " + file + " ignored");
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   291
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   292
            try {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   293
                if (verbose)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   294
                    errWriter.println(file);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   295
                fileCount.incrementAndGet();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   296
                NPETester p = new NPETester();
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   297
                readAndCheck(file, p::test);
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   298
            } catch (Throwable t) {
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   299
                if (!quiet) {
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   300
                    error("Error checking " + file + "\n" + t.getMessage());
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   301
                }
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   302
            }
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   303
        }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   304
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   305
        /**
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   306
         * Read and check a file.
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   307
         * @param file the file to be read
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   308
         * @return the tree for the content of the file
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   309
         * @throws IOException if any IO errors occur
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   310
         * @throws AttributionException if any errors occur while analyzing the file
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   311
         */
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   312
        void readAndCheck(File file, BiConsumer<JCCompilationUnit, JCTree> c) throws IOException {
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   313
            Iterable<? extends JavaFileObject> files = fileManager().getJavaFileObjects(file);
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   314
            final List<Element> analyzedElems = new ArrayList<>();
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   315
            final List<CompilationUnitTree> trees = new ArrayList<>();
47863
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   316
            totalNumberOfCompilations++;
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   317
            newCompilationTask()
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   318
                .withWriter(pw)
49518
d0ff431a596e 8198314: javac hidden options violate standard syntax for options
vromero
parents: 47863
diff changeset
   319
                    .withOption("--should-stop=at=ATTR")
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   320
                    .withOption("-XDverboseCompilePolicy")
47863
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   321
                    .withOption("-Xdoclint:none")
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   322
                    .withSource(files.iterator().next())
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   323
                    .withListener(new TaskListener() {
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   324
                        public void started(TaskEvent e) {
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   325
                            if (e.getKind() == TaskEvent.Kind.ANALYZE)
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   326
                            analyzedElems.add(e.getTypeElement());
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   327
                    }
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   328
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   329
                    public void finished(TaskEvent e) {
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   330
                        if (e.getKind() == Kind.PARSE)
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   331
                            trees.add(e.getCompilationUnit());
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   332
                    }
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   333
                }).analyze(res -> {
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   334
                Iterable<? extends Element> elems = res.get();
47863
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   335
                if (elems.iterator().hasNext()) {
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   336
                    for (CompilationUnitTree t : trees) {
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   337
                       JCCompilationUnit cu = (JCCompilationUnit)t;
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   338
                       for (JCTree def : cu.defs) {
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   339
                           if (def.hasTag(CLASSDEF) &&
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   340
                                   analyzedElems.contains(((JCTree.JCClassDecl)def).sym)) {
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   341
                               c.accept(cu, def);
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   342
                           }
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   343
                       }
47863
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   344
                    }
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   345
                } else {
abe64cd683a4 8190906: tools/javac/failover/CheckAttributedTree.java failed with timeout
vromero
parents: 47350
diff changeset
   346
                    numberOfFailedCompilations++;
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   347
                }
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   348
            });
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   349
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   350
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   351
        /**
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   352
         * Report an error. When the program is complete, the program will either
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   353
         * exit or throw an Error if any errors have been reported.
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   354
         * @param msg the error message
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   355
         */
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   356
        void error(String msg) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   357
            System.err.println();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   358
            System.err.println(msg);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   359
            System.err.println();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   360
            fail(msg);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   361
        }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   362
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   363
        /**
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   364
         * Main class for testing assertions concerning types/symbol
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   365
         * left uninitialized after attribution
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   366
         */
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   367
        private class NPETester extends TreeScanner {
47350
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   368
            void test(JCCompilationUnit cut, JCTree tree) {
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   369
                sourcefile = cut.sourcefile;
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   370
                endPosTable = cut.endPositions;
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   371
                encl = new Info(tree, endPosTable);
d65c3b21081c 8186694: JShell: speed-up compilation by reusing compiler instances
jlahoda
parents: 47216
diff changeset
   372
                tree.accept(this);
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   373
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   374
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   375
            @Override
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   376
            public void scan(JCTree tree) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   377
                if (tree == null ||
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   378
                        excludeTags.contains(treeUtil.nameFromTag(tree.getTag()))) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   379
                    return;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   380
                }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   381
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   382
                Info self = new Info(tree, endPosTable);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   383
                if (mandatoryType(tree)) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   384
                    check(tree.type != null,
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   385
                            "'null' field 'type' found in tree ", self);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   386
                    if (tree.type==null)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   387
                        Thread.dumpStack();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   388
                }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   389
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   390
                Field errField = checkFields(tree);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   391
                if (errField!=null) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   392
                    check(false,
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   393
                            "'null' field '" + errField.getName() + "' found in tree ", self);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   394
                }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   395
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   396
                Info prevEncl = encl;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   397
                encl = self;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   398
                tree.accept(this);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   399
                encl = prevEncl;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   400
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   401
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   402
            private boolean mandatoryType(JCTree that) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   403
                return that instanceof JCTree.JCExpression ||
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   404
                        that.hasTag(VARDEF) ||
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   405
                        that.hasTag(METHODDEF) ||
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   406
                        that.hasTag(CLASSDEF);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   407
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   408
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   409
            private final List<String> excludedFields = Arrays.asList("varargsElement", "targetType");
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   410
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   411
            void check(boolean ok, String label, Info self) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   412
                if (!ok) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   413
                    if (gui) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   414
                        if (viewer == null)
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   415
                            viewer = new Viewer();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   416
                        viewer.addEntry(sourcefile, label, encl, self);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   417
                    }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   418
                    error(label + self.toString() + " encl: " + encl.toString() +
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   419
                            " in file: " + sourcefile + "  " + self.tree);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   420
                }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   421
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   422
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   423
            Field checkFields(JCTree t) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   424
                List<Field> fieldsToCheck = treeUtil.getFieldsOfType(t,
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   425
                        excludedFields,
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   426
                        Symbol.class,
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   427
                        Type.class);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   428
                for (Field f : fieldsToCheck) {
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   429
                    try {
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   430
                        if (f.get(t) == null) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   431
                            return f;
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   432
                        }
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   433
                    }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   434
                    catch (IllegalAccessException e) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   435
                        System.err.println("Cannot read field: " + f);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   436
                        //swallow it
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   437
                    }
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   438
                }
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   439
                return null;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   440
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   441
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   442
            @Override
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   443
            public void visitImport(JCImport tree) { }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   444
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   445
            @Override
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   446
            public void visitTopLevel(JCCompilationUnit tree) {
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   447
                scan(tree.defs);
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   448
            }
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   449
51563
de411d537aae 8206986: Compiler support for Switch Expressions (Preview)
jlahoda
parents: 49518
diff changeset
   450
            @Override
de411d537aae 8206986: Compiler support for Switch Expressions (Preview)
jlahoda
parents: 49518
diff changeset
   451
            public void visitBreak(JCBreak tree) {
de411d537aae 8206986: Compiler support for Switch Expressions (Preview)
jlahoda
parents: 49518
diff changeset
   452
                if (tree.isValueBreak())
de411d537aae 8206986: Compiler support for Switch Expressions (Preview)
jlahoda
parents: 49518
diff changeset
   453
                    super.visitBreak(tree);
de411d537aae 8206986: Compiler support for Switch Expressions (Preview)
jlahoda
parents: 49518
diff changeset
   454
            }
de411d537aae 8206986: Compiler support for Switch Expressions (Preview)
jlahoda
parents: 49518
diff changeset
   455
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   456
            JavaFileObject sourcefile;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   457
            EndPosTable endPosTable;
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   458
            Info encl;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   459
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   460
    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   461
7846
f72ce57eb9c1 6982992: Tests CheckAttributedTree.java, JavacTreeScannerTest.java, and SourceTreeeScannerTest.java timeout
jjh
parents: 6601
diff changeset
   462
    // See CR:  6982992 Tests CheckAttributedTree.java, JavacTreeScannerTest.java, and SourceTreeeScannerTest.java timeout
f72ce57eb9c1 6982992: Tests CheckAttributedTree.java, JavacTreeScannerTest.java, and SourceTreeeScannerTest.java timeout
jjh
parents: 6601
diff changeset
   463
    StringWriter sw = new StringWriter();
f72ce57eb9c1 6982992: Tests CheckAttributedTree.java, JavacTreeScannerTest.java, and SourceTreeeScannerTest.java timeout
jjh
parents: 6601
diff changeset
   464
    PrintWriter pw = new PrintWriter(sw);
f72ce57eb9c1 6982992: Tests CheckAttributedTree.java, JavacTreeScannerTest.java, and SourceTreeeScannerTest.java timeout
jjh
parents: 6601
diff changeset
   465
32454
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   466
    StringWriter errSWriter = new StringWriter();
b0ac04e0fefe 8129962: Investigate performance improvements in langtools combo tests
mcimadamore
parents: 30730
diff changeset
   467
    PrintWriter errWriter = new PrintWriter(errSWriter);
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   468
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   469
    /** Flag: don't report irrelevant files. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   470
    boolean quiet;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   471
    /** Flag: show errors in GUI viewer. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   472
    boolean gui;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   473
    /** The GUI viewer for errors. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   474
    Viewer viewer;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   475
    /** Flag: report files as they are processed. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   476
    boolean verbose;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   477
    /** Option: encoding for test files. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   478
    String encoding;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   479
    /** The set of tags for tree nodes to be analyzed; if empty, all tree nodes
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   480
     * are analyzed. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   481
    Set<String> tags = new HashSet<String>();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   482
    /** Set of files and directories to be excluded from analysis. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   483
    Set<File> excludeFiles = new HashSet<File>();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   484
    /** Set of tag names to be excluded from analysis. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   485
    Set<String> excludeTags = new HashSet<String>();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   486
    /** Utility class for trees */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   487
    TreeUtil treeUtil = new TreeUtil();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   488
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   489
    /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   490
     * Utility class providing easy access to position and other info for a tree node.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   491
     */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   492
    private class Info {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   493
        Info() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   494
            tree = null;
10950
e87b50888909 6921494: provide way to print javac tree tag values
jjg
parents: 7846
diff changeset
   495
            tag = ERRONEOUS;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   496
            start = 0;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   497
            pos = 0;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   498
            end = Integer.MAX_VALUE;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   499
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   500
11055
ec1418effa77 7106166: (javac) re-factor EndPos parser
ksrini
parents: 10950
diff changeset
   501
        Info(JCTree tree, EndPosTable endPosTable) {
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   502
            this.tree = tree;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   503
            tag = tree.getTag();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   504
            start = TreeInfo.getStartPos(tree);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   505
            pos = tree.pos;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   506
            end = TreeInfo.getEndPos(tree, endPosTable);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   507
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   508
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   509
        @Override
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   510
        public String toString() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   511
            return treeUtil.nameFromTag(tree.getTag()) + "[start:" + start + ",pos:" + pos + ",end:" + end + "]";
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   512
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   513
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   514
        final JCTree tree;
10950
e87b50888909 6921494: provide way to print javac tree tag values
jjg
parents: 7846
diff changeset
   515
        final JCTree.Tag tag;
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   516
        final int start;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   517
        final int pos;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   518
        final int end;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   519
    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   520
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   521
    /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   522
     * Names for tree tags.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   523
     */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   524
    private static class TreeUtil {
10950
e87b50888909 6921494: provide way to print javac tree tag values
jjg
parents: 7846
diff changeset
   525
        String nameFromTag(JCTree.Tag tag) {
e87b50888909 6921494: provide way to print javac tree tag values
jjg
parents: 7846
diff changeset
   526
            String name = tag.name();
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   527
            return (name == null) ? "??" : name;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   528
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   529
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   530
        List<Field> getFieldsOfType(JCTree t, List<String> excludeNames, Class<?>... types) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   531
            List<Field> buf = new ArrayList<Field>();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   532
            for (Field f : t.getClass().getDeclaredFields()) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   533
                if (!excludeNames.contains(f.getName())) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   534
                    for (Class<?> type : types) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   535
                        if (type.isAssignableFrom(f.getType())) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   536
                            f.setAccessible(true);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   537
                            buf.add(f);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   538
                            break;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   539
                        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   540
                    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   541
                }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   542
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   543
            return buf;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   544
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   545
    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   546
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   547
    /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   548
     * GUI viewer for issues found by TreePosTester. The viewer provides a drop
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   549
     * down list for selecting error conditions, a header area providing details
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   550
     * about an error, and a text area with the ranges of text highlighted as
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   551
     * appropriate.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   552
     */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   553
    private class Viewer extends JFrame {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   554
        /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   555
         * Create a viewer.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   556
         */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   557
        Viewer() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   558
            initGUI();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   559
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   560
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   561
        /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   562
         * Add another entry to the list of errors.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   563
         * @param file The file containing the error
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   564
         * @param check The condition that was being tested, and which failed
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   565
         * @param encl the enclosing tree node
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   566
         * @param self the tree node containing the error
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   567
         */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   568
        void addEntry(JavaFileObject file, String check, Info encl, Info self) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   569
            Entry e = new Entry(file, check, encl, self);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   570
            DefaultComboBoxModel m = (DefaultComboBoxModel) entries.getModel();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   571
            m.addElement(e);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   572
            if (m.getSize() == 1)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   573
                entries.setSelectedItem(e);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   574
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   575
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   576
        /**
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   577
         * Initialize the GUI window.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   578
         */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   579
        private void initGUI() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   580
            JPanel head = new JPanel(new GridBagLayout());
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   581
            GridBagConstraints lc = new GridBagConstraints();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   582
            GridBagConstraints fc = new GridBagConstraints();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   583
            fc.anchor = GridBagConstraints.WEST;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   584
            fc.fill = GridBagConstraints.HORIZONTAL;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   585
            fc.gridwidth = GridBagConstraints.REMAINDER;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   586
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   587
            entries = new JComboBox();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   588
            entries.addActionListener(new ActionListener() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   589
                public void actionPerformed(ActionEvent e) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   590
                    showEntry((Entry) entries.getSelectedItem());
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   591
                }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   592
            });
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   593
            fc.insets.bottom = 10;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   594
            head.add(entries, fc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   595
            fc.insets.bottom = 0;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   596
            head.add(new JLabel("check:"), lc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   597
            head.add(checkField = createTextField(80), fc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   598
            fc.fill = GridBagConstraints.NONE;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   599
            head.add(setBackground(new JLabel("encl:"), enclColor), lc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   600
            head.add(enclPanel = new InfoPanel(), fc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   601
            head.add(setBackground(new JLabel("self:"), selfColor), lc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   602
            head.add(selfPanel = new InfoPanel(), fc);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   603
            add(head, BorderLayout.NORTH);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   604
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   605
            body = new JTextArea();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   606
            body.setFont(Font.decode(Font.MONOSPACED));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   607
            body.addCaretListener(new CaretListener() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   608
                public void caretUpdate(CaretEvent e) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   609
                    int dot = e.getDot();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   610
                    int mark = e.getMark();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   611
                    if (dot == mark)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   612
                        statusText.setText("dot: " + dot);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   613
                    else
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   614
                        statusText.setText("dot: " + dot + ", mark:" + mark);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   615
                }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   616
            });
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   617
            JScrollPane p = new JScrollPane(body,
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   618
                    JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   619
                    JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   620
            p.setPreferredSize(new Dimension(640, 480));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   621
            add(p, BorderLayout.CENTER);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   622
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   623
            statusText = createTextField(80);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   624
            add(statusText, BorderLayout.SOUTH);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   625
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   626
            pack();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   627
            setLocationRelativeTo(null); // centered on screen
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   628
            setVisible(true);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   629
            setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   630
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   631
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   632
        /** Show an entry that has been selected. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   633
        private void showEntry(Entry e) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   634
            try {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   635
                // update simple fields
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   636
                setTitle(e.file.getName());
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   637
                checkField.setText(e.check);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   638
                enclPanel.setInfo(e.encl);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   639
                selfPanel.setInfo(e.self);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   640
                // show file text with highlights
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   641
                body.setText(e.file.getCharContent(true).toString());
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   642
                Highlighter highlighter = body.getHighlighter();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   643
                highlighter.removeAllHighlights();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   644
                addHighlight(highlighter, e.encl, enclColor);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   645
                addHighlight(highlighter, e.self, selfColor);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   646
                scroll(body, getMinPos(enclPanel.info, selfPanel.info));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   647
            } catch (IOException ex) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   648
                body.setText("Cannot read " + e.file.getName() + ": " + e);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   649
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   650
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   651
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   652
        /** Create a test field. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   653
        private JTextField createTextField(int width) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   654
            JTextField f = new JTextField(width);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   655
            f.setEditable(false);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   656
            f.setBorder(null);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   657
            return f;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   658
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   659
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   660
        /** Add a highlighted region based on the positions in an Info object. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   661
        private void addHighlight(Highlighter h, Info info, Color c) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   662
            int start = info.start;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   663
            int end = info.end;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   664
            if (start == -1 && end == -1)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   665
                return;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   666
            if (start == -1)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   667
                start = end;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   668
            if (end == -1)
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   669
                end = start;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   670
            try {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   671
                h.addHighlight(info.start, info.end,
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   672
                        new DefaultHighlighter.DefaultHighlightPainter(c));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   673
                if (info.pos != -1) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   674
                    Color c2 = new Color(c.getRed(), c.getGreen(), c.getBlue(), (int)(.4f * 255)); // 40%
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   675
                    h.addHighlight(info.pos, info.pos + 1,
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   676
                        new DefaultHighlighter.DefaultHighlightPainter(c2));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   677
                }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   678
            } catch (BadLocationException e) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   679
                e.printStackTrace();
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   680
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   681
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   682
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   683
        /** Get the minimum valid position in a set of info objects. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   684
        private int getMinPos(Info... values) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   685
            int i = Integer.MAX_VALUE;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   686
            for (Info info: values) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   687
                if (info.start >= 0) i = Math.min(i, info.start);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   688
                if (info.pos   >= 0) i = Math.min(i, info.pos);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   689
                if (info.end   >= 0) i = Math.min(i, info.end);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   690
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   691
            return (i == Integer.MAX_VALUE) ? 0 : i;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   692
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   693
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   694
        /** Set the background on a component. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   695
        private JComponent setBackground(JComponent comp, Color c) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   696
            comp.setOpaque(true);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   697
            comp.setBackground(c);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   698
            return comp;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   699
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   700
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   701
        /** Scroll a text area to display a given position near the middle of the visible area. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   702
        private void scroll(final JTextArea t, final int pos) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   703
            // Using invokeLater appears to give text a chance to sort itself out
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   704
            // before the scroll happens; otherwise scrollRectToVisible doesn't work.
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   705
            // Maybe there's a better way to sync with the text...
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   706
            EventQueue.invokeLater(new Runnable() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   707
                public void run() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   708
                    try {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   709
                        Rectangle r = t.modelToView(pos);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   710
                        JScrollPane p = (JScrollPane) SwingUtilities.getAncestorOfClass(JScrollPane.class, t);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   711
                        r.y = Math.max(0, r.y - p.getHeight() * 2 / 5);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   712
                        r.height += p.getHeight() * 4 / 5;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   713
                        t.scrollRectToVisible(r);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   714
                    } catch (BadLocationException ignore) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   715
                    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   716
                }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   717
            });
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   718
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   719
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   720
        private JComboBox entries;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   721
        private JTextField checkField;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   722
        private InfoPanel enclPanel;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   723
        private InfoPanel selfPanel;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   724
        private JTextArea body;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   725
        private JTextField statusText;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   726
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   727
        private Color selfColor = new Color(0.f, 1.f, 0.f, 0.2f); // 20% green
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   728
        private Color enclColor = new Color(1.f, 0.f, 0.f, 0.2f); // 20% red
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   729
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   730
        /** Panel to display an Info object. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   731
        private class InfoPanel extends JPanel {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   732
            InfoPanel() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   733
                add(tagName = createTextField(20));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   734
                add(new JLabel("start:"));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   735
                add(addListener(start = createTextField(6)));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   736
                add(new JLabel("pos:"));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   737
                add(addListener(pos = createTextField(6)));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   738
                add(new JLabel("end:"));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   739
                add(addListener(end = createTextField(6)));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   740
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   741
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   742
            void setInfo(Info info) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   743
                this.info = info;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   744
                tagName.setText(treeUtil.nameFromTag(info.tag));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   745
                start.setText(String.valueOf(info.start));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   746
                pos.setText(String.valueOf(info.pos));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   747
                end.setText(String.valueOf(info.end));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   748
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   749
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   750
            JTextField addListener(final JTextField f) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   751
                f.addMouseListener(new MouseAdapter() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   752
                    @Override
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   753
                    public void mouseClicked(MouseEvent e) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   754
                        body.setCaretPosition(Integer.valueOf(f.getText()));
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   755
                        body.getCaret().setVisible(true);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   756
                    }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   757
                });
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   758
                return f;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   759
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   760
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   761
            Info info;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   762
            JTextField tagName;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   763
            JTextField start;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   764
            JTextField pos;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   765
            JTextField end;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   766
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   767
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   768
        /** Object to record information about an error to be displayed. */
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   769
        private class Entry {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   770
            Entry(JavaFileObject file, String check, Info encl, Info self) {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   771
                this.file = file;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   772
                this.check = check;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   773
                this.encl = encl;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   774
                this.self= self;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   775
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   776
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   777
            @Override
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   778
            public String toString() {
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   779
                return file.getName() + " " + check + " " + getMinPos(encl, self);
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   780
            }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   781
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   782
            final JavaFileObject file;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   783
            final String check;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   784
            final Info encl;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   785
            final Info self;
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   786
        }
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   787
    }
15040
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   788
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   789
    /** Number of files that have been analyzed. */
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   790
    static AtomicInteger fileCount = new AtomicInteger();
99fd9483d3f0 8005167: execution time of combo tests in javac should be improved
vromero
parents: 14058
diff changeset
   791
6594
d43f068fba19 6970584: Flow.java should be more error-friendly
mcimadamore
parents:
diff changeset
   792
}