better comments and clearer action branching

awslabs · Sep 24, 2024 · 76d7368 · 76d7368
1 parent 1ff2443
commit 76d7368
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/include/aws/io/tls_channel_handler.h b/include/aws/io/tls_channel_handler.h
@@ -235,8 +235,8 @@ struct aws_tls_ctx_options {
     struct aws_byte_buf pkcs12_password;
 
     /**
-     * The settings we expose controls to when adding items to the keychain using
-     * SecItem are contained within this struct.
+     * On iOS/tvOS the available settings when adding items to the keychain using
+     * SecItem are contained within this struct. This is NOT supported on MacOS.
      */
     struct aws_secitem_options *secitem_options;
 

diff --git a/source/tls_channel_handler.c b/source/tls_channel_handler.c
@@ -261,12 +261,12 @@ int aws_tls_ctx_options_set_keychain_path(
     }
 
     return AWS_OP_SUCCESS;
-#else
+#endif /* __APPLE__ || AWS_OS_IOS */
+
     (void)options;
     (void)keychain_path_cursor;
     AWS_LOGF_ERROR(AWS_LS_IO_TLS, "static: Keychain path can only be set on MacOS.");
     return aws_raise_error(AWS_ERROR_PLATFORM_NOT_SUPPORTED);
-#endif /* __APPLE__ && !AWS_OS_IOS*/
 }
 
 int aws_tls_ctx_options_init_client_mtls_from_system_path(