diff options
author | Nat Goodspeed <nat@lindenlab.com> | 2017-09-30 22:05:21 -0400 |
---|---|---|
committer | Nat Goodspeed <nat@lindenlab.com> | 2017-09-30 22:05:21 -0400 |
commit | 6b508cd9d45d8c74477efb0b97b1cbcccb714e78 (patch) | |
tree | c797abcc8a4d2a8a6b4f001fcc8dcfd412ce7694 /indra/llcorehttp/_httplibcurl.cpp | |
parent | 586d697475da239060abd8df030778fbdfb0cf51 (diff) |
MAINT-7081: Eliminate unused variable errors after new refactoring.
The new helper functions check_curl_easy_setopt() and
check_curl_multi_setopt() encapsulate the pervasive idiom:
code = curl_{easy,multi}_setopt(handle, option, arg);
check_curl_{easy,multi}_code(code, option);
But since each of these helper functions contains its own local CURL{,M}code
variable 'code', having a caller-scope variable reused for every such call is
no longer necessary -- in fact is no longer used at all. That produces a fatal
warning with MSVC. Get rid of those now-unused variables.
Diffstat (limited to 'indra/llcorehttp/_httplibcurl.cpp')
-rw-r--r-- | indra/llcorehttp/_httplibcurl.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/indra/llcorehttp/_httplibcurl.cpp b/indra/llcorehttp/_httplibcurl.cpp index 947a065d0a..abd304f6a5 100644 --- a/indra/llcorehttp/_httplibcurl.cpp +++ b/indra/llcorehttp/_httplibcurl.cpp @@ -470,7 +470,6 @@ void HttpLibcurl::policyUpdated(int policy_class) HttpPolicyClass & options(policy.getClassOptions(policy_class)); CURLM * multi_handle(mMultiHandles[policy_class]); - CURLMcode code; // Enable policy if stalled policy.stallPolicy(policy_class, false); |