C++20 not Building with VS2019

[esigo]

Fixes #1121 (issue)

Changes

Please provide a brief description of the changes here.

For significant contributions please make sure you have completed the following items:

• CHANGELOG.md updated for non-trivial changes
• Unit tests have been added
• Changes in public API reviewed

[codecov]

Codecov Report

Merging #1144 (8965523) into main (5aa6327) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1144   +/-   ##
=======================================
  Coverage   93.39%   93.39%           
=======================================
  Files         165      165           
  Lines        6229     6229           
=======================================
  Hits         5817     5817           
  Misses        412      412           

[lalitb]

Thanks for the fix. Do you know why it is passing in our C++ pipeline here. Probably std::result_of is only used in the case of specific exporters which don't get tested against C++20 in our CI.

[esigo]

Thanks for the fix. Do you know why it is passing in our C++ pipeline here. Probably std::result_of is only used in the case of specific exporters which don't get tested against C++20 in our CI.

It seems gcc has not removed std::result_of yet.