Fix PR and codacy comments

michaelbausor · michaelbausor · commit edfe20c682f4 · 2017-04-21T09:02:13.000-07:00
diff --git a/google-cloud-core/src/main/java/com/google/cloud/BaseWriteChannel.java b/google-cloud-core/src/main/java/com/google/cloud/BaseWriteChannel.java
@@ -335,13 +335,13 @@ protected List<ValueHolder> toStringHelper() {
     public String toString() {
       StringBuilder builder = new StringBuilder();
       builder.append(getClass().getSimpleName())
-          .append("{");
+          .append('{');
       String nextSeparator = "";
-      for(ValueHolder valueHolder : toStringHelper()) {
+      for (ValueHolder valueHolder : toStringHelper()) {
         builder.append(nextSeparator).append(valueHolder);
         nextSeparator = ", ";
       }
-      builder.append("}");
+      builder.append('}');
       return builder.toString();
     }
   }
diff --git a/google-cloud-logging/src/main/java/com/google/cloud/logging/LoggingImpl.java b/google-cloud-logging/src/main/java/com/google/cloud/logging/LoggingImpl.java
@@ -75,7 +75,6 @@
 import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ExecutionException;
-import javax.annotation.Nullable;
 
 class LoggingImpl extends BaseService<LoggingOptions> implements Logging {
 

Original file line number	Diff line number	Diff line change
`@@ -335,13 +335,13 @@ protected List<ValueHolder> toStringHelper() {`
`335`	`335`	`public String toString() {`
`336`	`336`	`StringBuilder builder = new StringBuilder();`
`337`	`337`	`builder.append(getClass().getSimpleName())`
`338`		`- .append("{");`
	`338`	`+ .append('{');`
`339`	`339`	`String nextSeparator = "";`
`340`		`- for(ValueHolder valueHolder : toStringHelper()) {`
	`340`	`+ for (ValueHolder valueHolder : toStringHelper()) {`
`341`	`341`	`builder.append(nextSeparator).append(valueHolder);`
`342`	`342`	`nextSeparator = ", ";`
`343`	`343`	`}`
`344`		`- builder.append("}");`
	`344`	`+ builder.append('}');`
`345`	`345`	`return builder.toString();`
`346`	`346`	`}`
`347`	`347`	`}`