aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGennadiy Civil <gennadiycivil@users.noreply.github.com>2018-03-05 13:51:15 -0500
committerGitHub <noreply@github.com>2018-03-05 13:51:15 -0500
commitc3a007f3cf1ac6d2aff1fde2119ab6862c57b2ef (patch)
tree13649c78c28b7eda0d5a0d6232ea9b3749cded82
parenta7ffeca97406195813604d6f5ea2d834019d9b07 (diff)
parent73bf41299d197296e2a7f07d6c7c5c330f9b99d8 (diff)
downloadgoogletest-c3a007f3cf1ac6d2aff1fde2119ab6862c57b2ef.tar.gz
googletest-c3a007f3cf1ac6d2aff1fde2119ab6862c57b2ef.tar.bz2
googletest-c3a007f3cf1ac6d2aff1fde2119ab6862c57b2ef.zip
Merge pull request #1489 from gennadiycivil/master
Merges-2
-rw-r--r--googlemock/src/gmock-internal-utils.cc14
-rw-r--r--googlemock/src/gmock.cc4
2 files changed, 14 insertions, 4 deletions
diff --git a/googlemock/src/gmock-internal-utils.cc b/googlemock/src/gmock-internal-utils.cc
index 658fa62d..20c5a8db 100644
--- a/googlemock/src/gmock-internal-utils.cc
+++ b/googlemock/src/gmock-internal-utils.cc
@@ -70,8 +70,8 @@ GTEST_API_ std::string JoinAsTuple(const Strings& fields) {
// words. Each maximum substring of the form [A-Za-z][a-z]*|\d+ is
// treated as one word. For example, both "FooBar123" and
// "foo_bar_123" are converted to "foo bar 123".
-GTEST_API_ string ConvertIdentifierNameToWords(const char* id_name) {
- string result;
+GTEST_API_ std::string ConvertIdentifierNameToWords(const char* id_name) {
+ std::string result;
char prev_char = '\0';
for (const char* p = id_name; *p != '\0'; prev_char = *(p++)) {
// We don't care about the current locale as the input is
@@ -188,5 +188,15 @@ GTEST_API_ void Log(LogSeverity severity, const std::string& message,
std::cout << ::std::flush;
}
+void IllegalDoDefault(const char* file, int line) {
+ internal::Assert(
+ false, file, line,
+ "You are using DoDefault() inside a composite action like "
+ "DoAll() or WithArgs(). This is not supported for technical "
+ "reasons. Please instead spell out the default action, or "
+ "assign the default action to an Action variable and use "
+ "the variable in various places.");
+}
+
} // namespace internal
} // namespace testing
diff --git a/googlemock/src/gmock.cc b/googlemock/src/gmock.cc
index 3c370510..2308168b 100644
--- a/googlemock/src/gmock.cc
+++ b/googlemock/src/gmock.cc
@@ -136,8 +136,8 @@ static bool ParseGoogleMockIntFlag(const char* str, const char* flag,
if (value_str == NULL) return false;
// Sets *value to the value of the flag.
- *value = atoi(value_str);
- return true;
+ return ParseInt32(Message() << "The value of flag --" << flag,
+ value_str, value);
}
// The internal implementation of InitGoogleMock().