[Asterisk-code-review] Revert "Test cel: Fails when DONT OPTIMIZE is off" (asterisk[13])

Chris Savinovich asteriskteam at digium.com
Thu Feb 14 10:53:43 CST 2019


Hello Richard Mudgett, Friendly Automation, George Joseph, Sean Bright, Joshua C. Colp,

I'd like you to do a code review. Please visit

    https://gerrit.asterisk.org/10996

to review the following change.


Change subject: Revert "Test_cel: Fails when DONT_OPTIMIZE is off"
......................................................................

Revert "Test_cel: Fails when DONT_OPTIMIZE is off"

This reverts commit 9b85882f04ed4b48e7bb318dbbb969d90159f13a.

Change-Id: I36cf159abb103502e48d6f8fd70d560d7f0bda56
---
M include/asterisk/strings.h
M main/strings.c
2 files changed, 13 insertions(+), 15 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/96/10996/1

diff --git a/include/asterisk/strings.h b/include/asterisk/strings.h
index 2d66716..85393b4 100644
--- a/include/asterisk/strings.h
+++ b/include/asterisk/strings.h
@@ -1257,7 +1257,19 @@
  *
  * \retval str for convenience
  */
-char *attribute_pure ast_str_to_lower(char *str);
+static force_inline char *attribute_pure ast_str_to_lower(char *str)
+{
+	char *str_orig = str;
+	if (!str) {
+		return str;
+	}
+
+	for (; *str; ++str) {
+		*str = tolower(*str);
+	}
+
+	return str_orig;
+}
 
 /*!
  * \brief Convert a string to all upper-case
diff --git a/main/strings.c b/main/strings.c
index 43fbca4..8102c3e 100644
--- a/main/strings.c
+++ b/main/strings.c
@@ -391,17 +391,3 @@
 
 	return start;
 }
-
-char *attribute_pure ast_str_to_lower(char *str)
-{
-	char *str_orig = str;
-	if (!str) {
-		return str;
-	}
-
-	for (; *str; ++str) {
-		*str = tolower(*str);
-	}
-
-	return str_orig;
-}

-- 
To view, visit https://gerrit.asterisk.org/10996
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: I36cf159abb103502e48d6f8fd70d560d7f0bda56
Gerrit-Change-Number: 10996
Gerrit-PatchSet: 1
Gerrit-Owner: Chris Savinovich <csavinovich at digium.com>
Gerrit-Reviewer: Friendly Automation (1000185)
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua C. Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190214/ded6fdaa/attachment.html>


More information about the asterisk-code-review mailing list