<p>pespin has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/15024">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">utils: share static buffer in osmo_str_to{lower,upper}()<br><br>Change-Id: I032803faa0e27c2efdff1ff276acabab95a8319a<br>---<br>M src/utils.c<br>1 file changed, 8 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/24/15024/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/utils.c b/src/utils.c</span><br><span>index 7364bfb..ea1de0f 100644</span><br><span>--- a/src/utils.c</span><br><span>+++ b/src/utils.c</span><br><span>@@ -42,6 +42,8 @@</span><br><span>  * \file utils.c */</span><br><span> </span><br><span> static __thread char namebuf[255];</span><br><span style="color: hsl(120, 100%, 40%);">+/* shared by osmo_str_tolower() and osmo_str_toupper() */</span><br><span style="color: hsl(120, 100%, 40%);">+static __thread char capsbuf[128];</span><br><span> </span><br><span> /*! get human-readable string for given value</span><br><span>  *  \param[in] vs Array of value_string tuples</span><br><span>@@ -901,16 +903,15 @@</span><br><span> /*! Convert a string to lowercase, using a static buffer.</span><br><span>  * The resulting string may be truncated if the internally used static buffer is shorter than src.</span><br><span>  * The internal buffer is at least 128 bytes long, i.e. guaranteed to hold at least 127 characters and a</span><br><span style="color: hsl(0, 100%, 40%);">- * terminating nul.</span><br><span style="color: hsl(120, 100%, 40%);">+ * terminating nul. The static buffer returned is shared with osmo_str_toupper().</span><br><span>  * See also osmo_str_tolower_buf().</span><br><span>  * \param[in] src  String to convert to lowercase.</span><br><span>  * \returns Resulting lowercase string in a static buffer, always nul terminated.</span><br><span>  */</span><br><span> const char *osmo_str_tolower(const char *src)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-   static __thread char buf[128];</span><br><span style="color: hsl(0, 100%, 40%);">-  osmo_str_tolower_buf(buf, sizeof(buf), src);</span><br><span style="color: hsl(0, 100%, 40%);">-    return buf;</span><br><span style="color: hsl(120, 100%, 40%);">+   osmo_str_tolower_buf(capsbuf, sizeof(capsbuf), src);</span><br><span style="color: hsl(120, 100%, 40%);">+  return capsbuf;</span><br><span> }</span><br><span> </span><br><span> /*! Convert a string to lowercase, dynamically allocating the output from given talloc context</span><br><span>@@ -960,16 +961,15 @@</span><br><span> /*! Convert a string to uppercase, using a static buffer.</span><br><span>  * The resulting string may be truncated if the internally used static buffer is shorter than src.</span><br><span>  * The internal buffer is at least 128 bytes long, i.e. guaranteed to hold at least 127 characters and a</span><br><span style="color: hsl(0, 100%, 40%);">- * terminating nul.</span><br><span style="color: hsl(120, 100%, 40%);">+ * terminating nul. The static buffer returned is shared with osmo_str_tolower().</span><br><span>  * See also osmo_str_toupper_buf().</span><br><span>  * \param[in] src  String to convert to uppercase.</span><br><span>  * \returns Resulting uppercase string in a static buffer, always nul terminated.</span><br><span>  */</span><br><span> const char *osmo_str_toupper(const char *src)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-  static __thread char buf[128];</span><br><span style="color: hsl(0, 100%, 40%);">-  osmo_str_toupper_buf(buf, sizeof(buf), src);</span><br><span style="color: hsl(0, 100%, 40%);">-    return buf;</span><br><span style="color: hsl(120, 100%, 40%);">+   osmo_str_toupper_buf(capsbuf, sizeof(capsbuf), src);</span><br><span style="color: hsl(120, 100%, 40%);">+  return capsbuf;</span><br><span> }</span><br><span> </span><br><span> /*! Convert a string to uppercase, dynamically allocating the output from given talloc context</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/15024">change 15024</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmocore/+/15024"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I032803faa0e27c2efdff1ff276acabab95a8319a </div>
<div style="display:none"> Gerrit-Change-Number: 15024 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>