Is there test coverage?

Is there test coverage?

I think the partial test coverage is (roughly) in libcxx test's std/containers/associative/from_range_associative_containers.h. The coverage doesn't seem complete though.

Perhaps we should add a test file to MSVC STL which covers copy/move ctors, corresponding allocator-extended ctors, and copy/move assignment operators of associative containers.

I expect that non-throwing path is implicitly covered somehow, I think it is enough if there's a test for throwing path

I mean I'm asking for the coverage for the guard, not for the unneeded conditional removal.

We didn't have test coverage for the _RERAISE before and the transformation is fairly straightforward so I'm willing to accept this PR as-is, but adding test coverage in a followup that exercises the guard would certainly be reasonable.

#2308 has an example of coverage

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

Thanks for eliminating yet another occurrence of the try-catch-rethrow pattern! 🐈 🐈‍⬛ 😸