Remove leftover TODOs (#3514)

Author: dagnir

codegen/src/main/java/software/amazon/awssdk/codegen/poet/rules/EndpointResolverInterceptorSpec.java

@@ -173,7 +173,7 @@ private MethodSpec ruleParams() {
                 case AWS_S3_CONTROL_USE_ARN_REGION:
                 // end of S3 specific builtins
                 case AWS_STS_USE_GLOBAL_ENDPOINT:
-                    // TODO: handle this
+                    // V2 doesn't support this, only regional endpoints
                     return;
                 default:
                     throw new RuntimeException("Don't know how to set built-in " + m.getBuiltInEnum());

services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/endpoints/UseGlobalEndpointResolver.java

@@ -24,12 +24,15 @@
 import software.amazon.awssdk.profiles.ProfileProperty;
 import software.amazon.awssdk.regions.Region;
 import software.amazon.awssdk.regions.ServiceMetadataAdvancedOption;
+import software.amazon.awssdk.regions.servicemetadata.EnhancedS3ServiceMetadata;
 import software.amazon.awssdk.utils.Lazy;
 import software.amazon.awssdk.utils.Logger;
 
 /**
- * Note: Most of the logic is copied from EnhancedS3ServiceMetadata
- * // TODO: should probably refactor that class to reuse the same logic
+ * Resolve the use global endpoint setting for S3.
+ * <p>
+ * This logic is identical to that in {@link EnhancedS3ServiceMetadata}, there's no good way to share it aside from creating a
+ * protected API that both the s3 and regions module consume.
  */
 @SdkInternalApi
 public class UseGlobalEndpointResolver {