From 5b0a31a43cb4a1b424404c49a175fb3a72c47900 Mon Sep 17 00:00:00 2001
From: "Henry H. Andrews" <andrews_henry@yahoo.com>
Date: Wed, 15 Jan 2025 12:18:31 -0800
Subject: [PATCH 1/2] Feedback from mkistler about contentEncoding

---
 src/oas.md | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/src/oas.md b/src/oas.md
index 4f68ac4eaf..59438ee612 100644
--- a/src/oas.md
+++ b/src/oas.md
@@ -1746,8 +1746,9 @@ requestBody:
           name:
             type: string
           icon:
-            # The default with "contentEncoding" is application/octet-stream,
-            # so we need to set image media type(s) in the Encoding Object.
+            # The default content type with "contentEncoding" present
+            # is application/octet-stream, # so we need to set the correct
+            # image media type(s) in the Encoding Object.
             type: string
             contentEncoding: base64url
   encoding:
@@ -1800,16 +1801,19 @@ requestBody:
       schema:
         type: object
         properties:
-          # default for a string without `contentEncoding` is `text/plain`
+          # default content type for a string without `contentEncoding`
+          # is `text/plain`
           id:
             type: string
             format: uuid
 
-          # default for a schema without `type` is `application/octet-stream`
+          # default content type for a schema without `type`
+          # is `application/octet-stream`
           profileImage: {}
 
-          # default for arrays is based on the type in the `items`
-          # subschema, which is an object, so `application/json`
+          # default content type for arrays is based on the type
+          # in the `items` subschema, which is an object here,
+          # so the default content type for each item is `application/json`
           addresses:
             type: array
             items:
@@ -1833,7 +1837,7 @@ requestBody:
             type: string
             format: uuid
 
-          # Encoding Object overrides the default `application/json`
+          # Encoding Object overrides the default `application/json` content type
           # for each item in the array with `application/xml; charset=utf-8`
           addresses:
             description: addresses in XML format

From 628e9711c5c40e53a60b27dc6ea042d24d674157 Mon Sep 17 00:00:00 2001
From: Ralf Handl <ralf.handl@sap.com>
Date: Thu, 16 Jan 2025 17:26:51 +0100
Subject: [PATCH 2/2] Apply suggestions from code review

Co-authored-by: Mike Kistler <mikekistler@microsoft.com>
---
 src/oas.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/oas.md b/src/oas.md
index 59438ee612..8f0257a318 100644
--- a/src/oas.md
+++ b/src/oas.md
@@ -1747,7 +1747,7 @@ requestBody:
             type: string
           icon:
             # The default content type with "contentEncoding" present
-            # is application/octet-stream, # so we need to set the correct
+            # is application/octet-stream, so we need to set the correct
             # image media type(s) in the Encoding Object.
             type: string
             contentEncoding: base64url