chronos38 / googlemock

Automatically exported from code.google.com/p/googlemock
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Make gmock_doctor compatible with the compiler messages from clang #121

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Currently, gmock_doctor only works with gcc.

Original issue reported on code.google.com by vladlosev on 2 Sep 2010 at 9:09

GoogleCodeExporter commented 9 years ago
Vlad, could you update this issue with the name of the disease that 
gmock_doctor cannot diagnose when using Clang?  Thanks.

Original comment by w...@google.com on 8 Sep 2010 at 7:03

GoogleCodeExporter commented 9 years ago
All diseases except the TTB (type in the templated base) disease are now 
diagnosed. TTB is left undiagnosed due to poor/insufficient diagnostics from 
Clang. I am still waiting on the Clang people to provide me with the bug # so 
updates can be tracked here.

Original comment by vladlosev on 8 Sep 2010 at 2:49

GoogleCodeExporter commented 9 years ago
I'm fine with gmock_doctor not diagnosing TTB with Clang.  I think the raw 
error messages generated by Clang are already fine, if not great, and there are 
far more important things to spend our time on than this issue.  Given that TTB 
is the only missing piece, I'd like to close the issue for now.  We can always 
re-open it *if* there are actual user requests.  Thanks.

Original comment by w...@google.com on 8 Sep 2010 at 4:50

GoogleCodeExporter commented 9 years ago
Fixed by Vlad in trunk.

Original comment by w...@google.com on 9 Sep 2010 at 6:36