small code cleanup / robustness

languagetool-org · Sep 13, 2015 · 40e01d1 · 40e01d1
1 parent 9b77786
commit 40e01d1
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/languagetool-gui-commons/src/main/java/org/languagetool/gui/AboutDialog.java b/languagetool-gui-commons/src/main/java/org/languagetool/gui/AboutDialog.java
@@ -119,7 +119,7 @@ public void hyperlinkUpdate(HyperlinkEvent e) {
 
   private String getMaintainers() {
     final TreeMap<String, Language> list = new TreeMap<>();
-    for (final Language lang : Languages.get()) {
+    for (Language lang : Languages.get()) {
       if (!lang.isVariant()) {
         if (lang.getMaintainers() != null) {
           list.put(messages.getString(lang.getShortName()), lang);
@@ -128,22 +128,25 @@ private String getMaintainers() {
     }
     final StringBuilder maintainersInfo = new StringBuilder();
     maintainersInfo.append("<table border=0 cellspacing=0 cellpadding=0>");
-    for(String lang : list.keySet()) {
+    for (String lang : list.keySet()) {
       maintainersInfo.append("<tr valign=\"top\"><td>");
       maintainersInfo.append(lang);
       maintainersInfo.append(":</td>");
       maintainersInfo.append("<td>&nbsp;</td>");
       maintainersInfo.append("<td>");
       int i = 0;
-      for (Contributor contributor : list.get(lang).getMaintainers()) {
-        if (i > 0) {
-          maintainersInfo.append(", ");
-          if (i % 3 == 0) {
-            maintainersInfo.append("<br>");
+      Contributor[] maintainers = list.get(lang).getMaintainers();
+      if (maintainers != null) {
+        for (Contributor contributor : maintainers) {
+          if (i > 0) {
+            maintainersInfo.append(", ");
+            if (i % 3 == 0) {
+              maintainersInfo.append("<br>");
+            }
           }
+          maintainersInfo.append(contributor.getName());
+          i++;
         }
-        maintainersInfo.append(contributor.getName());
-        i++;
       }
       maintainersInfo.append("</td></tr>");
     }