Merge pull request #119 from GoogleCloudPlatform/checkstyle

Checkstyle fixes for async-rest sample.

Author: lesv

.gitmodules

@@ -0,0 +1,3 @@
+[submodule "java-repo-tools"]
+	path = java-repo-tools
+	url = https://github.com/GoogleCloudPlatform/java-repo-tools.git

google-checks.xml

@@ -4,7 +4,14 @@
   <artifactId>managed-vms-async-rest</artifactId>
   <version>1.0.0-SNAPSHOT</version>
   <packaging>war</packaging>
-  <name>Example Async Rest Webapp</name>
+
+  <parent>
+    <artifactId>doc-samples</artifactId>
+    <groupId>com.google.cloud</groupId>
+    <version>1.0.0</version>
+    <relativePath>../..</relativePath>
+  </parent>
+
   <properties>
     <places.appkey>YOUR_PLACES_APP_KEY</places.appkey>
     <jetty.version>9.3.7.v20160115</jetty.version>
@@ -26,25 +33,24 @@
         <version>${jetty.version}</version>
         <configuration>
          <systemProperties>
-          <systemProperty> 
+          <systemProperty>
             <name>com.google.appengine.demos.asyncrest.appKey</name>
             <value>${places.appkey}</value>
-          </systemProperty> 
+          </systemProperty>
          </systemProperties>
         </configuration>
        </plugin>
        <plugin>
          <groupId>com.google.appengine</groupId>
          <artifactId>gcloud-maven-plugin</artifactId>
-         <version>2.0.9.96.v20160203</version>
+         <version>2.0.9.95.v20160203</version>
          <configuration>
-	    <gcloud_directory>/usr/local/google-cloud-sdk</gcloud_directory>
-            <verbosity>debug</verbosity>
-            <log_level>debug</log_level>
-	    <non_docker_mode>false</non_docker_mode>
-	    <docker_build>remote</docker_build>
+           <verbosity>debug</verbosity>
+           <log_level>debug</log_level>
+           <non_docker_mode>false</non_docker_mode>
+            <docker_build>remote</docker_build>
         </configuration>
-      </plugin>      
+      </plugin>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-war-plugin</artifactId>

java-repo-tools

@@ -1,3 +1,19 @@
+/*
+ * Copyright 2016 Google Inc. All Rights Reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 package com.google.appengine.demos;
 
 import java.io.IOException;
@@ -13,9 +29,8 @@
 public class DumpServlet extends HttpServlet {
 
   @Override
-  protected void doGet(HttpServletRequest request,
-          HttpServletResponse response) throws ServletException,
-          IOException {
+  protected void doGet(HttpServletRequest request, HttpServletResponse response)
+      throws ServletException, IOException {
     response.setContentType("text/html");
     response.setStatus(HttpServletResponse.SC_OK);
 
@@ -28,19 +43,42 @@ protected void doGet(HttpServletRequest request,
     out.printf("getServletContextName=%s%n", getServletContext().getServletContextName());
     out.printf("virtualServerName=%s%n", getServletContext().getVirtualServerName());
     out.printf("contextPath=%s%n", getServletContext().getContextPath());
-    out.printf("version=%d.%d%n", getServletContext().getMajorVersion(), getServletContext().getMinorVersion());
-    out.printf("effectiveVersion=%d.%d%n", getServletContext().getEffectiveMajorVersion(), getServletContext().getEffectiveMinorVersion());
+    out.printf(
+        "version=%d.%d%n",
+        getServletContext().getMajorVersion(),
+        getServletContext().getMinorVersion());
+    out.printf(
+        "effectiveVersion=%d.%d%n",
+        getServletContext().getEffectiveMajorVersion(),
+        getServletContext().getEffectiveMinorVersion());
     out.println("</pre>");
     out.println("<h2>Request Fields:</h2>");
     out.println("<pre>");
-    out.printf("remoteHost/Addr:port=%s/%s:%d%n", request.getRemoteHost(), request.getRemoteAddr(), request.getRemotePort());
-    out.printf("localName/Addr:port=%s/%s:%d%n", request.getLocalName(), request.getLocalAddr(), request.getLocalPort());
-    out.printf("scheme=%s method=%s protocol=%s%n", request.getScheme(), request.getMethod(), request.getProtocol());
+    out.printf(
+        "remoteHost/Addr:port=%s/%s:%d%n",
+        request.getRemoteHost(),
+        request.getRemoteAddr(),
+        request.getRemotePort());
+    out.printf(
+        "localName/Addr:port=%s/%s:%d%n",
+        request.getLocalName(),
+        request.getLocalAddr(),
+        request.getLocalPort());
+    out.printf(
+        "scheme=%s method=%s protocol=%s%n",
+        request.getScheme(),
+        request.getMethod(),
+        request.getProtocol());
     out.printf("serverName:serverPort=%s:%d%n", request.getServerName(), request.getServerPort());
     out.printf("requestURI=%s%n", request.getRequestURI());
     out.printf("requestURL=%s%n", request.getRequestURL().toString());
-    out.printf("contextPath|servletPath|pathInfo=%s|%s|%s%n", request.getContextPath(), request.getServletPath(), request.getPathInfo());
-    out.printf("session/new=%s/%b%n", request.getSession(true).getId(), request.getSession().isNew());
+    out.printf(
+        "contextPath|servletPath|pathInfo=%s|%s|%s%n",
+        request.getContextPath(),
+        request.getServletPath(),
+        request.getPathInfo());
+    out.printf(
+        "session/new=%s/%b%n", request.getSession(true).getId(), request.getSession().isNew());
     out.println("</pre>");
     out.println("<h2>Request Headers:</h2>");
     out.println("<pre>");