changeset 11378:7e297d001c30

option annotation processor ensures that a non-empty help message starts with a capital letter
author Doug Simon <doug.simon@oracle.com>
date Tue, 20 Aug 2013 16:09:26 +0200
parents 0fbc1e418c88
children 0942e34b6c7d
files graal/com.oracle.graal.options/src/com/oracle/graal/options/OptionProcessor.java
diffstat 1 files changed, 10 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.options/src/com/oracle/graal/options/OptionProcessor.java	Tue Aug 20 15:33:39 2013 +0200
+++ b/graal/com.oracle.graal.options/src/com/oracle/graal/options/OptionProcessor.java	Tue Aug 20 16:09:26 2013 +0200
@@ -87,6 +87,15 @@
             return;
         }
 
+        String help = annotation.help();
+        if (help.length() != 0) {
+            char firstChar = help.charAt(0);
+            if (!Character.isUpperCase(firstChar)) {
+                processingEnv.getMessager().printMessage(Kind.ERROR, "Option help text must start with upper case letter", element);
+                return;
+            }
+        }
+
         String optionName = annotation.name();
         if (optionName.equals("")) {
             optionName = fieldName;
@@ -118,7 +127,7 @@
             enclosing = enclosing.getEnclosingElement();
         }
 
-        info.options.add(new OptionInfo(optionName, annotation.help(), optionType, declaringClass, field));
+        info.options.add(new OptionInfo(optionName, help, optionType, declaringClass, field));
     }
 
     private void createFiles(OptionsInfo info) {