Wed, 12 Nov 2008 14:17:03 +0000
6768932: Add support for multiline diagnostics
Summary: Added basic support for multiline/tabular diagnostics
Reviewed-by: jjg
duke@1 | 1 | /* |
xdono@54 | 2 | * Copyright 1999-2008 Sun Microsystems, Inc. All Rights Reserved. |
duke@1 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
duke@1 | 4 | * |
duke@1 | 5 | * This code is free software; you can redistribute it and/or modify it |
duke@1 | 6 | * under the terms of the GNU General Public License version 2 only, as |
duke@1 | 7 | * published by the Free Software Foundation. Sun designates this |
duke@1 | 8 | * particular file as subject to the "Classpath" exception as provided |
duke@1 | 9 | * by Sun in the LICENSE file that accompanied this code. |
duke@1 | 10 | * |
duke@1 | 11 | * This code is distributed in the hope that it will be useful, but WITHOUT |
duke@1 | 12 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
duke@1 | 13 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
duke@1 | 14 | * version 2 for more details (a copy is included in the LICENSE file that |
duke@1 | 15 | * accompanied this code). |
duke@1 | 16 | * |
duke@1 | 17 | * You should have received a copy of the GNU General Public License version |
duke@1 | 18 | * 2 along with this work; if not, write to the Free Software Foundation, |
duke@1 | 19 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
duke@1 | 20 | * |
duke@1 | 21 | * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara, |
duke@1 | 22 | * CA 95054 USA or visit www.sun.com if you need additional information or |
duke@1 | 23 | * have any questions. |
duke@1 | 24 | */ |
duke@1 | 25 | |
duke@1 | 26 | package com.sun.tools.javac.util; |
duke@1 | 27 | |
duke@1 | 28 | import java.io.*; |
duke@1 | 29 | import java.util.HashSet; |
duke@1 | 30 | import java.util.Map; |
duke@1 | 31 | import java.util.Set; |
duke@1 | 32 | import javax.tools.DiagnosticListener; |
duke@1 | 33 | import javax.tools.JavaFileObject; |
jjg@50 | 34 | |
jjg@50 | 35 | import com.sun.tools.javac.file.JavacFileManager; |
duke@1 | 36 | import com.sun.tools.javac.tree.JCTree; |
mcimadamore@83 | 37 | import com.sun.tools.javac.api.DiagnosticFormatter; |
duke@1 | 38 | import com.sun.tools.javac.util.JCDiagnostic.DiagnosticPosition; |
duke@1 | 39 | import com.sun.tools.javac.util.JCDiagnostic.DiagnosticType; |
duke@1 | 40 | |
duke@1 | 41 | /** A class for error logs. Reports errors and warnings, and |
duke@1 | 42 | * keeps track of error numbers and positions. |
duke@1 | 43 | * |
duke@1 | 44 | * <p><b>This is NOT part of any API supported by Sun Microsystems. If |
duke@1 | 45 | * you write code that depends on this, you do so at your own risk. |
duke@1 | 46 | * This code and its internal interfaces are subject to change or |
duke@1 | 47 | * deletion without notice.</b> |
duke@1 | 48 | */ |
jjg@73 | 49 | public class Log extends AbstractLog { |
duke@1 | 50 | /** The context key for the log. */ |
duke@1 | 51 | public static final Context.Key<Log> logKey |
duke@1 | 52 | = new Context.Key<Log>(); |
duke@1 | 53 | |
duke@1 | 54 | /** The context key for the output PrintWriter. */ |
duke@1 | 55 | public static final Context.Key<PrintWriter> outKey = |
duke@1 | 56 | new Context.Key<PrintWriter>(); |
duke@1 | 57 | |
duke@1 | 58 | //@Deprecated |
duke@1 | 59 | public final PrintWriter errWriter; |
duke@1 | 60 | |
duke@1 | 61 | //@Deprecated |
duke@1 | 62 | public final PrintWriter warnWriter; |
duke@1 | 63 | |
duke@1 | 64 | //@Deprecated |
duke@1 | 65 | public final PrintWriter noticeWriter; |
duke@1 | 66 | |
duke@1 | 67 | /** The maximum number of errors/warnings that are reported. |
duke@1 | 68 | */ |
duke@1 | 69 | public final int MaxErrors; |
duke@1 | 70 | public final int MaxWarnings; |
duke@1 | 71 | |
duke@1 | 72 | /** Switch: prompt user on each error. |
duke@1 | 73 | */ |
duke@1 | 74 | public boolean promptOnError; |
duke@1 | 75 | |
duke@1 | 76 | /** Switch: emit warning messages. |
duke@1 | 77 | */ |
duke@1 | 78 | public boolean emitWarnings; |
duke@1 | 79 | |
duke@1 | 80 | /** Print stack trace on errors? |
duke@1 | 81 | */ |
duke@1 | 82 | public boolean dumpOnError; |
duke@1 | 83 | |
duke@1 | 84 | /** Print multiple errors for same source locations. |
duke@1 | 85 | */ |
duke@1 | 86 | public boolean multipleErrors; |
duke@1 | 87 | |
duke@1 | 88 | /** |
duke@1 | 89 | * Diagnostic listener, if provided through programmatic |
duke@1 | 90 | * interface to javac (JSR 199). |
duke@1 | 91 | */ |
duke@1 | 92 | protected DiagnosticListener<? super JavaFileObject> diagListener; |
jjg@73 | 93 | |
duke@1 | 94 | /** |
duke@1 | 95 | * Formatter for diagnostics |
duke@1 | 96 | */ |
mcimadamore@83 | 97 | private DiagnosticFormatter<JCDiagnostic> diagFormatter; |
duke@1 | 98 | |
mcimadamore@136 | 99 | /** |
mcimadamore@136 | 100 | * JavacMessages object used for localization |
mcimadamore@136 | 101 | */ |
mcimadamore@136 | 102 | private JavacMessages messages; |
mcimadamore@136 | 103 | |
duke@1 | 104 | /** Construct a log with given I/O redirections. |
duke@1 | 105 | */ |
duke@1 | 106 | @Deprecated |
duke@1 | 107 | protected Log(Context context, PrintWriter errWriter, PrintWriter warnWriter, PrintWriter noticeWriter) { |
jjg@73 | 108 | super(JCDiagnostic.Factory.instance(context)); |
duke@1 | 109 | context.put(logKey, this); |
duke@1 | 110 | this.errWriter = errWriter; |
duke@1 | 111 | this.warnWriter = warnWriter; |
duke@1 | 112 | this.noticeWriter = noticeWriter; |
duke@1 | 113 | |
duke@1 | 114 | Options options = Options.instance(context); |
duke@1 | 115 | this.dumpOnError = options.get("-doe") != null; |
duke@1 | 116 | this.promptOnError = options.get("-prompt") != null; |
duke@1 | 117 | this.emitWarnings = options.get("-Xlint:none") == null; |
duke@1 | 118 | this.MaxErrors = getIntOption(options, "-Xmaxerrs", 100); |
duke@1 | 119 | this.MaxWarnings = getIntOption(options, "-Xmaxwarns", 100); |
duke@1 | 120 | |
mcimadamore@83 | 121 | boolean rawDiagnostics = options.get("rawDiagnostics") != null; |
mcimadamore@136 | 122 | messages = JavacMessages.instance(context); |
mcimadamore@137 | 123 | this.diagFormatter = rawDiagnostics ? new RawDiagnosticFormatter(options) : |
mcimadamore@136 | 124 | new BasicDiagnosticFormatter(options, messages); |
duke@1 | 125 | @SuppressWarnings("unchecked") // FIXME |
duke@1 | 126 | DiagnosticListener<? super JavaFileObject> diagListener = |
duke@1 | 127 | context.get(DiagnosticListener.class); |
duke@1 | 128 | this.diagListener = diagListener; |
duke@1 | 129 | } |
duke@1 | 130 | // where |
duke@1 | 131 | private int getIntOption(Options options, String optionName, int defaultValue) { |
duke@1 | 132 | String s = options.get(optionName); |
duke@1 | 133 | try { |
duke@1 | 134 | if (s != null) return Integer.parseInt(s); |
duke@1 | 135 | } catch (NumberFormatException e) { |
duke@1 | 136 | // silently ignore ill-formed numbers |
duke@1 | 137 | } |
duke@1 | 138 | return defaultValue; |
duke@1 | 139 | } |
duke@1 | 140 | |
duke@1 | 141 | /** The default writer for diagnostics |
duke@1 | 142 | */ |
duke@1 | 143 | static final PrintWriter defaultWriter(Context context) { |
duke@1 | 144 | PrintWriter result = context.get(outKey); |
duke@1 | 145 | if (result == null) |
duke@1 | 146 | context.put(outKey, result = new PrintWriter(System.err)); |
duke@1 | 147 | return result; |
duke@1 | 148 | } |
duke@1 | 149 | |
duke@1 | 150 | /** Construct a log with default settings. |
duke@1 | 151 | */ |
duke@1 | 152 | protected Log(Context context) { |
duke@1 | 153 | this(context, defaultWriter(context)); |
duke@1 | 154 | } |
duke@1 | 155 | |
duke@1 | 156 | /** Construct a log with all output redirected. |
duke@1 | 157 | */ |
duke@1 | 158 | protected Log(Context context, PrintWriter defaultWriter) { |
duke@1 | 159 | this(context, defaultWriter, defaultWriter, defaultWriter); |
duke@1 | 160 | } |
duke@1 | 161 | |
duke@1 | 162 | /** Get the Log instance for this context. */ |
duke@1 | 163 | public static Log instance(Context context) { |
duke@1 | 164 | Log instance = context.get(logKey); |
duke@1 | 165 | if (instance == null) |
duke@1 | 166 | instance = new Log(context); |
duke@1 | 167 | return instance; |
duke@1 | 168 | } |
duke@1 | 169 | |
duke@1 | 170 | /** The number of errors encountered so far. |
duke@1 | 171 | */ |
duke@1 | 172 | public int nerrors = 0; |
duke@1 | 173 | |
duke@1 | 174 | /** The number of warnings encountered so far. |
duke@1 | 175 | */ |
duke@1 | 176 | public int nwarnings = 0; |
duke@1 | 177 | |
duke@1 | 178 | /** A set of all errors generated so far. This is used to avoid printing an |
duke@1 | 179 | * error message more than once. For each error, a pair consisting of the |
duke@1 | 180 | * source file name and source code position of the error is added to the set. |
duke@1 | 181 | */ |
duke@1 | 182 | private Set<Pair<JavaFileObject, Integer>> recorded = new HashSet<Pair<JavaFileObject,Integer>>(); |
duke@1 | 183 | |
duke@1 | 184 | public boolean hasDiagnosticListener() { |
duke@1 | 185 | return diagListener != null; |
duke@1 | 186 | } |
duke@1 | 187 | |
duke@1 | 188 | public void setEndPosTable(JavaFileObject name, Map<JCTree, Integer> table) { |
jjg@73 | 189 | name.getClass(); // null check |
jjg@73 | 190 | getSource(name).setEndPosTable(table); |
duke@1 | 191 | } |
duke@1 | 192 | |
mcimadamore@168 | 193 | /** Return current sourcefile. |
duke@1 | 194 | */ |
mcimadamore@168 | 195 | public JavaFileObject currentSourceFile() { |
duke@1 | 196 | return source == null ? null : source.getFile(); |
duke@1 | 197 | } |
duke@1 | 198 | |
duke@1 | 199 | /** Flush the logs |
duke@1 | 200 | */ |
duke@1 | 201 | public void flush() { |
duke@1 | 202 | errWriter.flush(); |
duke@1 | 203 | warnWriter.flush(); |
duke@1 | 204 | noticeWriter.flush(); |
duke@1 | 205 | } |
duke@1 | 206 | |
duke@1 | 207 | /** Returns true if an error needs to be reported for a given |
duke@1 | 208 | * source name and pos. |
duke@1 | 209 | */ |
duke@1 | 210 | protected boolean shouldReport(JavaFileObject file, int pos) { |
duke@1 | 211 | if (multipleErrors || file == null) |
duke@1 | 212 | return true; |
duke@1 | 213 | |
duke@1 | 214 | Pair<JavaFileObject,Integer> coords = new Pair<JavaFileObject,Integer>(file, pos); |
duke@1 | 215 | boolean shouldReport = !recorded.contains(coords); |
duke@1 | 216 | if (shouldReport) |
duke@1 | 217 | recorded.add(coords); |
duke@1 | 218 | return shouldReport; |
duke@1 | 219 | } |
duke@1 | 220 | |
duke@1 | 221 | /** Prompt user after an error. |
duke@1 | 222 | */ |
duke@1 | 223 | public void prompt() { |
duke@1 | 224 | if (promptOnError) { |
duke@1 | 225 | System.err.println(getLocalizedString("resume.abort")); |
duke@1 | 226 | char ch; |
duke@1 | 227 | try { |
duke@1 | 228 | while (true) { |
duke@1 | 229 | switch (System.in.read()) { |
duke@1 | 230 | case 'a': case 'A': |
duke@1 | 231 | System.exit(-1); |
duke@1 | 232 | return; |
duke@1 | 233 | case 'r': case 'R': |
duke@1 | 234 | return; |
duke@1 | 235 | case 'x': case 'X': |
duke@1 | 236 | throw new AssertionError("user abort"); |
duke@1 | 237 | default: |
duke@1 | 238 | } |
duke@1 | 239 | } |
duke@1 | 240 | } catch (IOException e) {} |
duke@1 | 241 | } |
duke@1 | 242 | } |
duke@1 | 243 | |
duke@1 | 244 | /** Print the faulty source code line and point to the error. |
duke@1 | 245 | * @param pos Buffer index of the error position, must be on current line |
duke@1 | 246 | */ |
duke@1 | 247 | private void printErrLine(int pos, PrintWriter writer) { |
jjg@73 | 248 | String line = (source == null ? null : source.getLine(pos)); |
jjg@73 | 249 | if (line == null) |
duke@1 | 250 | return; |
mcimadamore@100 | 251 | int col = source.getColumnNumber(pos, false); |
duke@1 | 252 | |
jjg@73 | 253 | printLines(writer, line); |
jjg@73 | 254 | for (int i = 0; i < col - 1; i++) { |
jjg@73 | 255 | writer.print((line.charAt(i) == '\t') ? "\t" : " "); |
duke@1 | 256 | } |
duke@1 | 257 | writer.println("^"); |
duke@1 | 258 | writer.flush(); |
duke@1 | 259 | } |
duke@1 | 260 | |
duke@1 | 261 | /** Print the text of a message, translating newlines appropriately |
duke@1 | 262 | * for the platform. |
duke@1 | 263 | */ |
duke@1 | 264 | public static void printLines(PrintWriter writer, String msg) { |
duke@1 | 265 | int nl; |
duke@1 | 266 | while ((nl = msg.indexOf('\n')) != -1) { |
duke@1 | 267 | writer.println(msg.substring(0, nl)); |
duke@1 | 268 | msg = msg.substring(nl+1); |
duke@1 | 269 | } |
duke@1 | 270 | if (msg.length() != 0) writer.println(msg); |
duke@1 | 271 | } |
duke@1 | 272 | |
jjg@73 | 273 | protected void directError(String key, Object... args) { |
jjg@73 | 274 | printLines(errWriter, getLocalizedString(key, args)); |
jjg@73 | 275 | errWriter.flush(); |
duke@1 | 276 | } |
duke@1 | 277 | |
duke@1 | 278 | /** Report a warning that cannot be suppressed. |
duke@1 | 279 | * @param pos The source position at which to report the warning. |
duke@1 | 280 | * @param key The key for the localized warning message. |
duke@1 | 281 | * @param args Fields of the warning message. |
duke@1 | 282 | */ |
duke@1 | 283 | public void strictWarning(DiagnosticPosition pos, String key, Object ... args) { |
duke@1 | 284 | writeDiagnostic(diags.warning(source, pos, key, args)); |
duke@1 | 285 | nwarnings++; |
duke@1 | 286 | } |
duke@1 | 287 | |
duke@1 | 288 | /** |
duke@1 | 289 | * Common diagnostic handling. |
duke@1 | 290 | * The diagnostic is counted, and depending on the options and how many diagnostics have been |
duke@1 | 291 | * reported so far, the diagnostic may be handed off to writeDiagnostic. |
duke@1 | 292 | */ |
duke@1 | 293 | public void report(JCDiagnostic diagnostic) { |
duke@1 | 294 | switch (diagnostic.getType()) { |
duke@1 | 295 | case FRAGMENT: |
duke@1 | 296 | throw new IllegalArgumentException(); |
duke@1 | 297 | |
duke@1 | 298 | case NOTE: |
duke@1 | 299 | // Print out notes only when we are permitted to report warnings |
duke@1 | 300 | // Notes are only generated at the end of a compilation, so should be small |
duke@1 | 301 | // in number. |
duke@1 | 302 | if (emitWarnings || diagnostic.isMandatory()) { |
duke@1 | 303 | writeDiagnostic(diagnostic); |
duke@1 | 304 | } |
duke@1 | 305 | break; |
duke@1 | 306 | |
duke@1 | 307 | case WARNING: |
duke@1 | 308 | if (emitWarnings || diagnostic.isMandatory()) { |
duke@1 | 309 | if (nwarnings < MaxWarnings) { |
duke@1 | 310 | writeDiagnostic(diagnostic); |
duke@1 | 311 | nwarnings++; |
duke@1 | 312 | } |
duke@1 | 313 | } |
duke@1 | 314 | break; |
duke@1 | 315 | |
duke@1 | 316 | case ERROR: |
duke@1 | 317 | if (nerrors < MaxErrors |
duke@1 | 318 | && shouldReport(diagnostic.getSource(), diagnostic.getIntPosition())) { |
duke@1 | 319 | writeDiagnostic(diagnostic); |
duke@1 | 320 | nerrors++; |
duke@1 | 321 | } |
duke@1 | 322 | break; |
duke@1 | 323 | } |
duke@1 | 324 | } |
duke@1 | 325 | |
duke@1 | 326 | /** |
duke@1 | 327 | * Write out a diagnostic. |
duke@1 | 328 | */ |
duke@1 | 329 | protected void writeDiagnostic(JCDiagnostic diag) { |
duke@1 | 330 | if (diagListener != null) { |
duke@1 | 331 | try { |
duke@1 | 332 | diagListener.report(diag); |
duke@1 | 333 | return; |
duke@1 | 334 | } |
duke@1 | 335 | catch (Throwable t) { |
duke@1 | 336 | throw new ClientCodeException(t); |
duke@1 | 337 | } |
duke@1 | 338 | } |
duke@1 | 339 | |
duke@1 | 340 | PrintWriter writer = getWriterForDiagnosticType(diag.getType()); |
duke@1 | 341 | |
mcimadamore@136 | 342 | printLines(writer, diagFormatter.format(diag, messages.getCurrentLocale())); |
duke@1 | 343 | |
duke@1 | 344 | if (promptOnError) { |
duke@1 | 345 | switch (diag.getType()) { |
duke@1 | 346 | case ERROR: |
duke@1 | 347 | case WARNING: |
duke@1 | 348 | prompt(); |
duke@1 | 349 | } |
duke@1 | 350 | } |
duke@1 | 351 | |
duke@1 | 352 | if (dumpOnError) |
duke@1 | 353 | new RuntimeException().printStackTrace(writer); |
duke@1 | 354 | |
duke@1 | 355 | writer.flush(); |
duke@1 | 356 | } |
duke@1 | 357 | |
duke@1 | 358 | @Deprecated |
duke@1 | 359 | protected PrintWriter getWriterForDiagnosticType(DiagnosticType dt) { |
duke@1 | 360 | switch (dt) { |
duke@1 | 361 | case FRAGMENT: |
duke@1 | 362 | throw new IllegalArgumentException(); |
duke@1 | 363 | |
duke@1 | 364 | case NOTE: |
duke@1 | 365 | return noticeWriter; |
duke@1 | 366 | |
duke@1 | 367 | case WARNING: |
duke@1 | 368 | return warnWriter; |
duke@1 | 369 | |
duke@1 | 370 | case ERROR: |
duke@1 | 371 | return errWriter; |
duke@1 | 372 | |
duke@1 | 373 | default: |
duke@1 | 374 | throw new Error(); |
duke@1 | 375 | } |
duke@1 | 376 | } |
duke@1 | 377 | |
duke@1 | 378 | /** Find a localized string in the resource bundle. |
duke@1 | 379 | * @param key The key for the localized string. |
duke@1 | 380 | * @param args Fields to substitute into the string. |
duke@1 | 381 | */ |
duke@1 | 382 | public static String getLocalizedString(String key, Object ... args) { |
mcimadamore@136 | 383 | return JavacMessages.getDefaultLocalizedString("compiler.misc." + key, args); |
duke@1 | 384 | } |
duke@1 | 385 | |
duke@1 | 386 | /*************************************************************************** |
duke@1 | 387 | * raw error messages without internationalization; used for experimentation |
duke@1 | 388 | * and quick prototyping |
duke@1 | 389 | ***************************************************************************/ |
duke@1 | 390 | |
jjg@73 | 391 | /** print an error or warning message: |
jjg@73 | 392 | */ |
duke@1 | 393 | private void printRawError(int pos, String msg) { |
jjg@73 | 394 | if (source == null || pos == Position.NOPOS) { |
duke@1 | 395 | printLines(errWriter, "error: " + msg); |
duke@1 | 396 | } else { |
jjg@73 | 397 | int line = source.getLineNumber(pos); |
mcimadamore@168 | 398 | JavaFileObject file = source.getFile(); |
duke@1 | 399 | if (file != null) |
duke@1 | 400 | printLines(errWriter, |
duke@1 | 401 | JavacFileManager.getJavacFileName(file) + ":" + |
duke@1 | 402 | line + ": " + msg); |
duke@1 | 403 | printErrLine(pos, errWriter); |
duke@1 | 404 | } |
duke@1 | 405 | errWriter.flush(); |
duke@1 | 406 | } |
duke@1 | 407 | |
jjg@73 | 408 | /** report an error: |
jjg@73 | 409 | */ |
duke@1 | 410 | public void rawError(int pos, String msg) { |
mcimadamore@168 | 411 | if (nerrors < MaxErrors && shouldReport(currentSourceFile(), pos)) { |
duke@1 | 412 | printRawError(pos, msg); |
duke@1 | 413 | prompt(); |
duke@1 | 414 | nerrors++; |
duke@1 | 415 | } |
duke@1 | 416 | errWriter.flush(); |
duke@1 | 417 | } |
duke@1 | 418 | |
jjg@73 | 419 | /** report a warning: |
jjg@73 | 420 | */ |
duke@1 | 421 | public void rawWarning(int pos, String msg) { |
duke@1 | 422 | if (nwarnings < MaxWarnings && emitWarnings) { |
duke@1 | 423 | printRawError(pos, "warning: " + msg); |
duke@1 | 424 | } |
duke@1 | 425 | prompt(); |
duke@1 | 426 | nwarnings++; |
duke@1 | 427 | errWriter.flush(); |
duke@1 | 428 | } |
duke@1 | 429 | |
duke@1 | 430 | public static String format(String fmt, Object... args) { |
duke@1 | 431 | return String.format((java.util.Locale)null, fmt, args); |
duke@1 | 432 | } |
duke@1 | 433 | |
duke@1 | 434 | } |