[Asterisk-code-review] utils: In Solaris, avoid a warning about an unused variable. (asterisk[13])
Alexander Traud
asteriskteam at digium.com
Wed Mar 7 09:33:46 CST 2018
Alexander Traud has uploaded this change for review. ( https://gerrit.asterisk.org/8457
Change subject: utils: In Solaris, avoid a warning about an unused variable.
......................................................................
utils: In Solaris, avoid a warning about an unused variable.
When HAVE_GETHOSTBYNAME_R_5 was set by the script ./configure, GCC 7.3.0 found
an unused variable. Actually, the variable was used (set to a dummy value) but
the compiler optimization might have removed that. Instead, this change ensures
that the variable 'res' is only used when it is really required.
Change-Id: Ic3ea23ccf84ac4bc2d501b514985b989030abab5
---
M main/utils.c
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/57/8457/1
diff --git a/main/utils.c b/main/utils.c
index b4ecffd..56bcdd9 100644
--- a/main/utils.c
+++ b/main/utils.c
@@ -196,7 +196,9 @@
*/
struct hostent *ast_gethostbyname(const char *host, struct ast_hostent *hp)
{
+#ifndef HAVE_GETHOSTBYNAME_R_5
int res;
+#endif
int herrno;
int dots = 0;
const char *s;
@@ -206,7 +208,6 @@
integers, we break with tradition and refuse to look up a
pure integer */
s = host;
- res = 0;
while (s && *s) {
if (*s == '.')
dots++;
--
To view, visit https://gerrit.asterisk.org/8457
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic3ea23ccf84ac4bc2d501b514985b989030abab5
Gerrit-Change-Number: 8457
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180307/a29bedcc/attachment.html>
More information about the asterisk-code-review
mailing list