Remove redundant return from collectBodyString and isSerializableHeaderValue (#176)

Author: trivikr

smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/integration/HttpProtocolGeneratorUtils.java

@@ -226,11 +226,8 @@ static void generateCollectBodyString(GenerationContext context) {
 
         writer.addImport("SerdeContext", "__SerdeContext", "@aws-sdk/types");
         writer.write("// Encode Uint8Array data into string with utf-8.");
-        writer.openBlock("const collectBodyString = (streamBody: any, context: __SerdeContext): Promise<string> => {",
-                "};", () -> {
-            writer.write("return collectBody(streamBody, context).then(body => context.utf8Encoder(body));");
-        });
-
+        writer.write("const collectBodyString = (streamBody: any, context: __SerdeContext): Promise<string> => " +
+            "collectBody(streamBody, context).then(body => context.utf8Encoder(body))");
         writer.write("");
     }
 

smithy-typescript-codegen/src/main/resources/software/amazon/smithy/typescript/codegen/integration/http-binding-utils.ts

@@ -1,6 +1,6 @@
-function isSerializableHeaderValue(value: any): boolean {
-  return value !== undefined
-      && value !== ""
-      && (!Object.getOwnPropertyNames(value).includes("length") || value.length != 0)
-      && (!Object.getOwnPropertyNames(value).includes("size") || value.size != 0);
-}
+const isSerializableHeaderValue = (value: any): boolean =>
+  value !== undefined &&
+  value !== "" &&
+  (!Object.getOwnPropertyNames(value).includes("length") ||
+    value.length != 0) &&
+  (!Object.getOwnPropertyNames(value).includes("size") || value.size != 0);