Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove custom implementations of std::remove_reference #2395

Closed

Conversation

kuzkry
Copy link
Contributor

@kuzkry kuzkry commented Aug 14, 2019

@asoffer
Copy link
Contributor

asoffer commented Aug 14, 2019

263417291

@asoffer
Copy link
Contributor

asoffer commented Aug 16, 2019

It looks like we need to do some internal cleanup to get this working. I'll get that started and update when we're ready to merge this.

@kuzkry
Copy link
Contributor Author

kuzkry commented Aug 16, 2019

Ok, thanks! The problem here is all these type traits either depend on each other or were defined in similar places.

@kuzkry kuzkry force-pushed the custom-type-traits-remove_reference branch from efaf9e4 to 69651d1 Compare August 20, 2019 10:26
@kuzkry kuzkry force-pushed the custom-type-traits-remove_reference branch 2 times, most recently from f429c48 to 366e34c Compare August 22, 2019 07:53
@kuzkry
Copy link
Contributor Author

kuzkry commented Aug 22, 2019

Retriggering CI

@kuzkry kuzkry closed this Aug 22, 2019
@kuzkry kuzkry reopened this Aug 22, 2019
@kuzkry kuzkry force-pushed the custom-type-traits-remove_reference branch from 366e34c to 62f4b03 Compare August 24, 2019 10:30
@kuzkry kuzkry force-pushed the custom-type-traits-remove_reference branch from 62f4b03 to 5d7cdbb Compare August 27, 2019 22:02
rogeeff pushed a commit that referenced this pull request Sep 6, 2019
Merge 7f4f58d into 90a443f

Closes #2395

COPYBARA_INTEGRATE_REVIEW=#2395 from kuzkry:custom-type-traits-remove_reference 7f4f58d
PiperOrigin-RevId: 266189044
@rogeeff rogeeff closed this in ab8f346 Sep 6, 2019
kuzkry added a commit to kuzkry/googletest that referenced this pull request Sep 7, 2019
ab8f346 is a manual merge and it abandoned some things from PR google#2395.
@kuzkry kuzkry deleted the custom-type-traits-remove_reference branch September 7, 2019 04:03
kuzkry added a commit to kuzkry/googletest that referenced this pull request Sep 7, 2019
This fixes up ab8f346 (a manual merge) that has abandoned some things
from PR google#2395.
kuzkry added a commit to kuzkry/googletest that referenced this pull request Sep 16, 2019
This fixes up ab8f346 (a manual merge) that has abandoned some things
from PR google#2395.
thejcannon pushed a commit to thejcannon/googletest that referenced this pull request Oct 23, 2019
This fixes up ab8f346 (a manual merge) that has abandoned some things
from PR google#2395.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants