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