commit 09cca313d2f39b149551a6cc9758934fbf927506
author David Pursehouse <david.pursehouse@sonymobile.com> Mon Dec 03 11:17:34 2012 +0900
committer David Pursehouse <david.pursehouse@sonymobile.com> Mon Dec 03 11:38:50 2012 +0900
tree d3230242c2cc4a8f66bf79e3ef26425cba2c2cf1
parent ec32548cdc7e4cef2b0579c9db2896592e040ee1

Fix compiler warnings about missing switch cases for AuthType

As in 272d5c9 add explicit case statements to silence the
compiler warnings.

Change-Id: I37323409ea793eacd8249d4e9f477a66ef927049

gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyIdentitiesScreen.java

diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyIdentitiesScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyIdentitiesScreen.java
index 899aa03..79a6bca 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyIdentitiesScreen.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/account/MyIdentitiesScreen.java
@@ -72,6 +72,16 @@
         add(linkIdentity);
         break;
       }
+
+      case CLIENT_SSL_CERT_LDAP:
+      case CUSTOM_EXTENSION:
+      case DEVELOPMENT_BECOME_ANY_ACCOUNT:
+      case HTTP:
+      case HTTP_LDAP:
+      case LDAP:
+      case LDAP_BIND:
+      case OPENID_SSO:
+        break;
     }
   }
 

gerrit-httpd/src/main/java/com/google/gerrit/httpd/GerritConfigProvider.java

diff --git a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/GerritConfigProvider.java b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/GerritConfigProvider.java
index 2e26256..9b81f0d 100644
--- a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/GerritConfigProvider.java
+++ b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/GerritConfigProvider.java
@@ -107,6 +107,12 @@
         config.setEditFullNameUrl(cfg.getString("auth", null, "editFullNameUrl"));
         config.setHttpPasswordUrl(cfg.getString("auth", null, "httpPasswordUrl"));
         break;
+
+      case CLIENT_SSL_CERT_LDAP:
+      case DEVELOPMENT_BECOME_ANY_ACCOUNT:
+      case HTTP:
+      case HTTP_LDAP:
+        break;
     }
     config.setUseContributorAgreements(cfg.getBoolean("auth",
         "contributoragreements", false));

gerrit-pgm/src/main/java/com/google/gerrit/pgm/init/InitAuth.java

diff --git a/gerrit-pgm/src/main/java/com/google/gerrit/pgm/init/InitAuth.java b/gerrit-pgm/src/main/java/com/google/gerrit/pgm/init/InitAuth.java
index ea12008..1524707 100644
--- a/gerrit-pgm/src/main/java/com/google/gerrit/pgm/init/InitAuth.java
+++ b/gerrit-pgm/src/main/java/com/google/gerrit/pgm/init/InitAuth.java
@@ -54,6 +54,15 @@
         auth.string("SSO logout URL", "logoutUrl", null);
         break;
       }
+
+      case CLIENT_SSL_CERT_LDAP:
+      case CUSTOM_EXTENSION:
+      case DEVELOPMENT_BECOME_ANY_ACCOUNT:
+      case LDAP:
+      case LDAP_BIND:
+      case OPENID:
+      case OPENID_SSO:
+        break;
     }
 
     switch (auth_type) {
@@ -80,6 +89,14 @@
         ldap.string("Group BaseDN", "groupBase", aBase);
         break;
       }
+
+      case CLIENT_SSL_CERT_LDAP:
+      case CUSTOM_EXTENSION:
+      case DEVELOPMENT_BECOME_ANY_ACCOUNT:
+      case HTTP:
+      case OPENID:
+      case OPENID_SSO:
+        break;
     }
 
     if (auth.getSecure("registerEmailPrivateKey") == null) {