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