Apply suggestions from code review

disneystreaming · Aug 8, 2023 · 83d6d3c · 83d6d3c
1 parent b60496d
commit 83d6d3c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/modules/core/src/smithy4s/http/internals/UrlFormDataDecoderSchemaVisitor.scala b/modules/core/src/smithy4s/http/internals/UrlFormDataDecoderSchemaVisitor.scala
@@ -66,13 +66,13 @@ private[smithy4s] class UrlFormDataDecoderSchemaVisitor(
 
         case UrlFormCursor(history, values) =>
           // Collection members aren't necessarily primitives, and if they
-          // aren't, then there will be multiple pathed values for the same
+          // aren't, then there will be multiple values for the same
           // index. One example is maps, which are encoded as collections of
           // structs, e.g.
           // foos.entry.1.key=a&foos.entry.1.value=1&foos.entry.2.key=b&foos.entry.2.value=2.
           // That's why we have to group by index.
           //
-          // We can't assume they were encoding in order. That's why we have to
+          // We can't assume they were encoded in order. That's why we have to
           // then sort by index.
           val groupedAndSortedCursors = values
             .collect {

diff --git a/modules/core/src/smithy4s/http/internals/UrlFormDataEncoderSchemaVisitor.scala b/modules/core/src/smithy4s/http/internals/UrlFormDataEncoderSchemaVisitor.scala
@@ -39,7 +39,7 @@ private[smithy4s] class UrlFormDataEncoderSchemaVisitor(
   ): UrlFormDataEncoder[P] =
     Primitive.stringWriter(tag, hints) match {
       case Some(writer) =>
-        p => List(UrlForm.FormData(PayloadPath.root, Some(writer(p))))
+        primitive => List(UrlForm.FormData(PayloadPath.root, Some(writer(primitive))))
 
       case None =>
         _ => Nil