Remove unnecessary type casting

[compnerd]

The compiler now imports these as their canonical type, allowing us to directly use without the explicit type cast.

[compnerd]

@swift-ci please test

[owenv]

Do we need to stage this in somehow? Unfortunately a lot of the dependents still have swift 6.2 CI jobs

[compnerd]

@owenv no, this should be backwards compatible. I'm not sure why the Linux build is impacted by this, this should only be touching the Windows paths. The WinSDK+Overlay.swift is the backwards compatibility. Once we can move to 6.5, we should be able to drop that.

[compnerd]

@swift-ci please test

[compnerd]

@swift-ci please test Windows platform

[owenv]

ok, I think this should be fine then so long as the tests pass

[owenv]

The linux failures are happening because we stopped producing toolchains for AL2 and the workflow YAML is out of date. UB9 is failing because the latest dev snapshot skipped it when publishing. Not caused by this change

[compnerd]

@swift-ci please test

[compnerd]

@swift-ci please test Windows platform

[compnerd]

@swift-ci please test Linux platform

[compnerd]

@swift-ci please test Linux platform

[compnerd]

23:46:57  /home/build-user/swift-argument-parser/Sources/ArgumentParser/Parsing/CommandParser.swift:305:11: error: consecutive declarations on a line must be separated by ';'
23:46:57    ) throws(CommandError) -> ParsableCommand {
23:46:57            ^
23:46:57            ;

Seems like an infrastructure issue on Linux @owenv @jakepetroules

[owenv]

This job is using Swift 5.9 for some reason, I'm looking at why

[compnerd]

@swift-ci please test Linux platform