googletest
googletest copied to clipboard
work around GCC 6~11 ADL bug
see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51577 ADL seems to work properly when we do the SFINAE check via the return type, but not when using a dummy template parameter
fix #3992
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
Generally looks good. Please sign the CLA, and we'll get this merged.
The CLA should be signed, but somehow it still doesn't work. The issue is being discussed here: https://github.com/google/closure-compiler/pull/3988 In the meantime I will update the PR to use trailing return type as you requested.
@asoffer Sorry for taking so long, but the CLA check is finally green now.
@asoffer Hi. I made the changes you requested and the CLA check is green. Is there anything blocking us from moving forward with this?
@asoffer @derekmauro I'd really like to proceed with this. Please let me know if there's something I need to do before this can be merged.
@asoffer @derekmauro @dinord could someone of you please look into this? This bug is what is keeping me from moving to following HEAD of googletest in my projects as they fail to compile :eyes:
is there any news on this one?
Unfortunately not and I have the feeling that my pings to @asoffer @derekmauro and @dinord didn't get through :shrug: