Skip to content

Commit

Permalink
url: mark ignored return value in node::url::Parse(...)
Browse files Browse the repository at this point in the history
This is to resolve an unused result warning in node_url.cc.
Resolve macro redefinition warning on Windows

PR-URL: #10141
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
  • Loading branch information
Christopher J. Brody authored and evanlucas committed Jan 4, 2017
1 parent f9a5c13 commit 495213e
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions src/node_url.cc
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ using v8::Value;
url.flags |= URL_FLAGS_TERMINATED; \
goto done; \
}
#define FAILED() \
#define URL_FAILED() \
{ \
url.flags |= URL_FLAGS_FAILED; \
goto done; \
Expand Down Expand Up @@ -744,7 +744,7 @@ namespace url {
break;
case kNoScheme:
if (!has_base || (IS_CANNOT_BE_BASE(base.flags) && ch != '#')) {
FAILED()
URL_FAILED()
} else if (IS_CANNOT_BE_BASE(base.flags) && ch == '#') {
SET_HAVE_SCHEME()
url.scheme = base.scheme;
Expand Down Expand Up @@ -982,10 +982,10 @@ namespace url {
case kHostname:
if (ch == ':' && !sbflag) {
if (special && buffer.size() == 0)
FAILED()
URL_FAILED()
SET_HAVE_HOST()
if (ParseHost(&buffer, &url.host) < 0)
FAILED()
URL_FAILED()
buffer.clear();
state = kPort;
if (override == kHostname)
Expand All @@ -997,10 +997,10 @@ namespace url {
special_back_slash) {
p--;
if (special && buffer.size() == 0)
FAILED()
URL_FAILED()
SET_HAVE_HOST()
if (ParseHost(&buffer, &url.host) < 0)
FAILED()
URL_FAILED()
buffer.clear();
state = kPathStart;
if (state_override)
Expand Down Expand Up @@ -1029,14 +1029,14 @@ namespace url {
if (port >= 0 && port <= 0xffff) {
url.port = NormalizePort(url.scheme, port);
} else if (!state_override) {
FAILED()
URL_FAILED()
}
buffer.clear();
}
state = kPathStart;
continue;
} else {
FAILED();
URL_FAILED();
}
break;
case kFile:
Expand Down Expand Up @@ -1151,7 +1151,7 @@ namespace url {
if (buffer != "localhost") {
SET_HAVE_HOST()
if (ParseHost(&buffer, &url.host) < 0)
FAILED()
URL_FAILED()
}
buffer.clear();
state = kPathStart;
Expand Down Expand Up @@ -1294,7 +1294,7 @@ namespace url {
argv[ARG_PATH] = Copy(isolate, url.path);
}

cb->Call(context, recv, 9, argv);
(void)cb->Call(context, recv, 9, argv);
}

static void Parse(const FunctionCallbackInfo<Value>& args) {
Expand Down

0 comments on commit 495213e

Please sign in to comment.