Compilation of dependent target fails with G++ targeting C++ standard

[luxapana]

I am using latest release tag.
I get following compiler error when trying to include headers:

include/opentelemetry/sdk/metrics/sync_instruments.h:192:25: error: expected unqualified-id before ‘)’ token
    BoundUpDownCounter<T>() = default;

Apparently according to C++ 20 standard, above syntax is illegal.

We are using multiple features of C++ 20 so reverting to 17 would not work.

Any suggestions?

[lalitb]

Apparently according to C++ 20 standard, above syntax is illegal.

Which version of g++ compiler? I don't see this construct deprecated in C++17, or removed in C++ 20 ( http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2020/p2131r0.html#removed ).

Any suggestions?

Feel free to raise a PR for the fix, and we will review the change - if that helps?

[luxapana]

Here it is:
https://timsong-cpp.github.io/cppwp/n4861/diff.cpp17.class#2

Can certainly raise a PR for this.

[lalitb]

https://timsong-cpp.github.io/cppwp/n4861/diff.cpp17.class#2

Thank you for the link. Will look forward to the PR.

[lalitb]

#1029 fixes it.