JDK 14 RFR of JDK-8230626: Make UnknownFooException strings more informative
Joe Darcy
joe.darcy at oracle.com
Wed Sep 4 22:20:51 UTC 2019
Updated webrev with patch below:
http://cr.openjdk.java.net/~darcy/8230626.1/
-Joe
---
old/src/java.compiler/share/classes/javax/lang/model/element/UnknownAnnotationValueException.java
2019-09-04 15:19:08.561963468 -0700
+++
new/src/java.compiler/share/classes/javax/lang/model/element/UnknownAnnotationValueException.java
2019-09-04 15:19:08.385963468 -0700
@@ -60,7 +60,7 @@
* @param p an additional parameter, may be {@code null}
*/
public UnknownAnnotationValueException(AnnotationValue av, Object p) {
- super("Unknown annotation value: " + av);
+ super("Unknown annotation value: \"" + av + "\"");
this.av = av;
this.parameter = p;
}
---
old/src/java.compiler/share/classes/javax/lang/model/element/UnknownElementException.java
2019-09-04 15:19:09.005963468 -0700
+++
new/src/java.compiler/share/classes/javax/lang/model/element/UnknownElementException.java
2019-09-04 15:19:08.837963468 -0700
@@ -58,7 +58,7 @@
* @param p an additional parameter, may be {@code null}
*/
public UnknownElementException(Element e, Object p) {
- super("Unknown element: " + e);
+ super("Unknown element: \"" + e + "\"");
element = e;
this.parameter = p;
}
---
old/src/java.compiler/share/classes/javax/lang/model/type/UnknownTypeException.java
2019-09-04 15:19:09.509963468 -0700
+++
new/src/java.compiler/share/classes/javax/lang/model/type/UnknownTypeException.java
2019-09-04 15:19:09.305963468 -0700
@@ -58,7 +58,7 @@
* @param p an additional parameter, may be {@code null}
*/
public UnknownTypeException(TypeMirror t, Object p) {
- super("Unknown type: " + t);
+ super("Unknown type: \"" + t + "\"");
type = t;
this.parameter = p;
}
On 9/4/2019 3:04 PM, mark.reinhold at oracle.com wrote:
> 2019/9/4 14:19:00 -0700, joe.darcy at oracle.com:
>> Responding to some off-list input from Jon, please review the small
>> enhancement for
>>
>> JDK-8230626: Make UnknownFooException strings more informative
>> http://cr.openjdk.java.net/~darcy/8230626.0/
>>
>> Patch below; thanks,
>> ...
>> - super("Unknown annotation value: " + av);
>> + super("Unknown annotation value: ``" + av + "''");
> The use of GNU-style ``fake curly quotes'' in JDK messages is extremely
> rare, and jarring to the eye in most typefaces. Pairs of backticks and
> apostrophes are a poorer subsitute for “true curly quotes” than simple
> "vertical quotation marks." Please use vertical quotation marks, which
> is what we use pretty much everywhere.
>
> - Mark
More information about the compiler-dev
mailing list