[Japi-cvs] SF.net SVN: japi:[880] libs/swing-action/trunk/src/prj/net/sf/japi/swing

[<chr...@us...>]

Revision: 880
          http://japi.svn.sourceforge.net/japi/?rev=880&view=rev
Author:   christianhujer
Date:     2009-02-15 12:31:16 +0000 (Sun, 15 Feb 2009)

Log Message:
-----------
 Fixed checkstyle issues. Includes change of all TODO comments into a format with date.

Modified Paths:
--------------
    libs/swing-action/trunk/src/prj/net/sf/japi/swing/DefaultActionBuilder.java
    libs/swing-action/trunk/src/prj/net/sf/japi/swing/ReflectionAction.java
    libs/swing-action/trunk/src/prj/net/sf/japi/swing/ToggleAction.java

Modified: libs/swing-action/trunk/src/prj/net/sf/japi/swing/DefaultActionBuilder.java
===================================================================
--- libs/swing-action/trunk/src/prj/net/sf/japi/swing/DefaultActionBuilder.java	2009-02-15 12:31:01 UTC (rev 879)
+++ libs/swing-action/trunk/src/prj/net/sf/japi/swing/DefaultActionBuilder.java	2009-02-15 12:31:16 UTC (rev 880)
@@ -852,7 +852,7 @@
             return JOptionPane.PLAIN_MESSAGE;
         }
         if (!typeText.endsWith("_MESSAGE")) {
-            System.err.println("Warning: Illegal type value " + typeText + " for dialog " + dialogKey); // TODO:christianhujer:I18N/L10N
+            System.err.println("Warning: Illegal type value " + typeText + " for dialog " + dialogKey); // TODO:2009-02-15:christianhujer:I18N/L10N
             return JOptionPane.PLAIN_MESSAGE;
         }
         if ("PLAIN_MESSAGE".equals(typeText)) {
@@ -873,11 +873,11 @@
                     return f.getInt(null);
                 }
             } catch (final NoSuchFieldException e) {
-                System.err.println("Warning: Field " + typeText + " not found in JOptionPane (dialog: " + dialogKey + ")."); // TODO:christianhujer:I18N/L10N
-                e.printStackTrace();  //TODO:christianhujer:Replace with proper logging.
+                System.err.println("Warning: Field " + typeText + " not found in JOptionPane (dialog: " + dialogKey + ")."); // TODO:2009-02-15:christianhujer:I18N/L10N
+                e.printStackTrace();  //TODO:2009-02-15:christianhujer:Replace with proper logging.
             } catch (final IllegalAccessException e) {
-                System.err.println("Warning: Field " + typeText + " not accessible in JOptionPane (dialog: " + dialogKey + ")."); // TODO:christianhujer:I18N/L10N
-                e.printStackTrace();  //TODO:christianhujer:Replace with proper logging.
+                System.err.println("Warning: Field " + typeText + " not accessible in JOptionPane (dialog: " + dialogKey + ")."); // TODO:2009-02-15:christianhujer:I18N/L10N
+                e.printStackTrace();  //TODO:2009-02-15:christianhujer:Replace with proper logging.
             }
             return JOptionPane.PLAIN_MESSAGE;
         }

Modified: libs/swing-action/trunk/src/prj/net/sf/japi/swing/ReflectionAction.java
===================================================================
--- libs/swing-action/trunk/src/prj/net/sf/japi/swing/ReflectionAction.java	2009-02-15 12:31:01 UTC (rev 879)
+++ libs/swing-action/trunk/src/prj/net/sf/japi/swing/ReflectionAction.java	2009-02-15 12:31:16 UTC (rev 880)
@@ -166,13 +166,13 @@
         if (!isEnabled()) { return; }
         final Object instance = getValue(REFLECTION_TARGET);
         try {
-            // TODO:christianhujer:Special value for REFLECTION_ARGUMENTS which determines whether to use ActionEvent.
+            // TODO:2009-02-15:christianhujer:Special value for REFLECTION_ARGUMENTS which determines whether to use ActionEvent.
             final Method method = getMethod(instance);
             if (method != null) {
                 final Object[] arguments = getArguments(method, e);
                 method.invoke(instance, arguments);
             } else {
-                // TODO:christianhujer:Improve logging / debugging
+                // TODO:2009-02-15:christianhujer:Improve logging / debugging
                 System.err.println("Missing implementation for action " + getValue(ACTION_ID));
             }
         } catch (final IllegalAccessException ex) {
@@ -187,7 +187,7 @@
                     if (title != null) {
                         // source cannot be null because the ActionEvent constructor will not allow a null source.
                         @NotNull final Object source = e.getSource();
-                        final Component parent = source instanceof Component ? (Component) source : null; // TODO:christianhujer:find better alternative to null
+                        final Component parent = source instanceof Component ? (Component) source : null; // TODO:2009-02-15:christianhujer:find better alternative to null
                         actionBuilder.showMessageDialog(parent, dialogKey, cause.getLocalizedMessage());
                         return;
                     }

Modified: libs/swing-action/trunk/src/prj/net/sf/japi/swing/ToggleAction.java
===================================================================
--- libs/swing-action/trunk/src/prj/net/sf/japi/swing/ToggleAction.java	2009-02-15 12:31:01 UTC (rev 879)
+++ libs/swing-action/trunk/src/prj/net/sf/japi/swing/ToggleAction.java	2009-02-15 12:31:16 UTC (rev 880)
@@ -87,7 +87,7 @@
         } catch (final IllegalAccessException ex) {
             assert false : ex;
         } catch (final InvocationTargetException ex) {
-            // XXX:christianhujer:workaround bug in IntelliJ IDEA (ex is NOT ignored)
+            // XXX:2009-02-15:christianhujer:workaround bug in IntelliJ IDEA (ex is NOT ignored)
             //noinspection ThrowInsideCatchBlockWhichIgnoresCaughtException
             ex.printStackTrace();
             throw new RuntimeException(ex.getCause());


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.