remove commented code

xinsong-cui · xinsong-cui · commit ca01936893e4 · 2025-01-07T17:02:47.000-05:00
diff --git a/codegen/aws-sdk-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/ServiceClientCompanionObjectWriter.kt b/codegen/aws-sdk-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/ServiceClientCompanionObjectWriter.kt
@@ -35,7 +35,6 @@ class ServiceClientCompanionObjectWriter : AppendingSectionWriter {
             declareSection(FinalizeEnvironmentalConfig) {
                 write("super.#L(builder, sharedConfig, activeProfile)", funName)
                 writeResolveEndpointUrl()
-//                writeRegionWithRegionProvider()
             }
         }
     }
@@ -55,16 +54,6 @@ class ServiceClientCompanionObjectWriter : AppendingSectionWriter {
             write("#S,", names.sharedConfigKey)
         }
     }
-
-//    private fun KotlinWriter.writeRegionWithRegionProvider() {
-//        withBlock(
-//            "builder.config.region = builder.config.region ?: builder.config.regionProvider?.getRegion() ?: #T(",
-//            ")",
-//            AwsRuntimeTypes.Config.Region.resolveRegion,
-//        ) {
-//            write("profile = activeProfile")
-//        }
-//    }
 }
 
 internal data class EndpointUrlConfigNames(

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,6 @@ class ServiceClientCompanionObjectWriter : AppendingSectionWriter {`
`35`	`35`	`declareSection(FinalizeEnvironmentalConfig) {`
`36`	`36`	`write("super.#L(builder, sharedConfig, activeProfile)", funName)`
`37`	`37`	`writeResolveEndpointUrl()`
`38`		`-// writeRegionWithRegionProvider()`
`39`	`38`	`}`
`40`	`39`	`}`
`41`	`40`	`}`
`@@ -55,16 +54,6 @@ class ServiceClientCompanionObjectWriter : AppendingSectionWriter {`
`55`	`54`	`write("#S,", names.sharedConfigKey)`
`56`	`55`	`}`
`57`	`56`	`}`
`58`		`-`
`59`		`-// private fun KotlinWriter.writeRegionWithRegionProvider() {`
`60`		`-// withBlock(`
`61`		`-// "builder.config.region = builder.config.region ?: builder.config.regionProvider?.getRegion() ?: #T(",`
`62`		`-// ")",`
`63`		`-// AwsRuntimeTypes.Config.Region.resolveRegion,`
`64`		`-// ) {`
`65`		`-// write("profile = activeProfile")`
`66`		`-// }`
`67`		`-// }`
`68`	`57`	`}`
`69`	`58`
`70`	`59`	`internal data class EndpointUrlConfigNames(`