creamidea / googlemock

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

Patch for /trunk/include/gmock/gmock-matchers.h #152

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Removed extra ; cause gcc compiler warning with pedantic flag
This is for inline Matcher<T> MakeMatcher(const MatcherInterface<T>* impl)

Original issue reported on code.google.com by borydbe...@gmail.com on 10 Jan 2012 at 2:31

Attachments:

GoogleCodeExporter commented 8 years ago
Please consider using this fix soon. It is very easy, but quite annoying. We 
have to change the sources in our local repository, outherwise our projects 
will not build.

Original comment by matthias...@ptvgroup.com on 8 Mar 2013 at 7:34

GoogleCodeExporter commented 8 years ago
Reasonable.  I'll get this fixed.

In general, I suggest to compile your project with -isystem 
PATH/TO/GMOCK/HEADERS though, such that you aren't bothered by warnings in 
googlemock that has nothing to do with your own code.

Original comment by w...@google.com on 8 Mar 2013 at 5:23

GoogleCodeExporter commented 8 years ago

Original comment by w...@google.com on 8 Mar 2013 at 5:25

GoogleCodeExporter commented 8 years ago
Done in trunk (r425).

Original comment by w...@google.com on 8 Mar 2013 at 5:54