Removes TODOs that aren't needed (#2698)

Author: cenedhryn

core/auth-crt/src/main/java/software/amazon/awssdk/authcrt/signer/internal/CrtHttpRequestConverter.java

@@ -147,15 +147,13 @@ private HttpHeader[] createHttpHeaderArray(SdkHttpFullRequest request) {
         return crtHeaderList.toArray(new HttpHeader[0]);
     }
 
-    //TODO When CRT can work with SDK format empty paths, this method can be removed
     private static String encodedPathToCrtFormat(String sdkEncodedPath) {
         if (StringUtils.isEmpty(sdkEncodedPath)) {
             return "/";
         }
         return sdkEncodedPath;
     }
 
-    //TODO When CRT can work with SDK empty paths, this method can be removed
     private static String encodedPathFromCrtFormat(String sdkEncodedPath, String crtEncodedPath) {
         if (SLASH.equals(crtEncodedPath) && StringUtils.isEmpty(sdkEncodedPath)) {
             return "";

core/aws-core/src/main/java/software/amazon/awssdk/awscore/internal/AwsExecutionContextBuilder.java

@@ -56,8 +56,8 @@ private AwsExecutionContextBuilder() {
     public static <InputT extends SdkRequest, OutputT extends SdkResponse> ExecutionContext
         invokeInterceptorsAndCreateExecutionContext(ClientExecutionParams<InputT, OutputT> executionParams,
                                                     SdkClientConfiguration clientConfig) {
-        // Note: This is currently copied to DefaultS3Presigner and other presigners. Don't edit this without considering those
-        // as well. TODO: Probably don't copy all of this manually.
+        // Note: This is currently copied to DefaultS3Presigner and other presigners.
+        // Don't edit this without considering those
 
         SdkRequest originalRequest = executionParams.getInput();
         MetricCollector metricCollector = resolveMetricCollector(executionParams);

core/sdk-core/src/main/java/software/amazon/awssdk/core/internal/handler/BaseClientHandler.java

@@ -171,8 +171,7 @@ private static InterceptorContext runModifyHttpRequestAndHttpContentInterceptors
                     (OutputT) response.toBuilder().sdkHttpResponse(httpFullResponse).build());
     }
 
-    //TODO: Remove/throw exception when called. This method is only called from tests, since the subclasses
-    // in aws-core override it.
+    // This method is only called from tests, since the subclasses in aws-core override it.
     protected <InputT extends SdkRequest, OutputT extends SdkResponse> ExecutionContext
         invokeInterceptorsAndCreateExecutionContext(
         ClientExecutionParams<InputT, OutputT> params) {