Title: [152087] trunk/Source/WebCore
Revision
152087
Author
commit-qu...@webkit.org
Date
2013-06-27 03:28:07 -0700 (Thu, 27 Jun 2013)

Log Message

[curl] Fix unused variable/parameter warnings
https://bugs.webkit.org/show_bug.cgi?id=118050

Patch by Peter Gal <galpe...@inf.u-szeged.hu> on 2013-06-27
Reviewed by Christophe Dumez.

Commented out unused parameters and removed an unused variable in the curl backend.

No tests required.

* platform/network/curl/CredentialStorageCurl.cpp:
(WebCore::CredentialStorage::getFromPersistentStorage):
* platform/network/curl/DNSCurl.cpp:
(WebCore::prefetchDNS):
* platform/network/curl/ResourceHandleManager.cpp:
(WebCore::curl_lock_callback):
(WebCore::curl_unlock_callback):
(WebCore::headerCallback):
(WebCore::ResourceHandleManager::downloadTimerCallback):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (152086 => 152087)


--- trunk/Source/WebCore/ChangeLog	2013-06-27 10:10:08 UTC (rev 152086)
+++ trunk/Source/WebCore/ChangeLog	2013-06-27 10:28:07 UTC (rev 152087)
@@ -1,3 +1,24 @@
+2013-06-27  Peter Gal  <galpe...@inf.u-szeged.hu>
+
+        [curl] Fix unused variable/parameter warnings
+        https://bugs.webkit.org/show_bug.cgi?id=118050
+
+        Reviewed by Christophe Dumez.
+
+        Commented out unused parameters and removed an unused variable in the curl backend.
+
+        No tests required.
+
+        * platform/network/curl/CredentialStorageCurl.cpp:
+        (WebCore::CredentialStorage::getFromPersistentStorage):
+        * platform/network/curl/DNSCurl.cpp:
+        (WebCore::prefetchDNS):
+        * platform/network/curl/ResourceHandleManager.cpp:
+        (WebCore::curl_lock_callback):
+        (WebCore::curl_unlock_callback):
+        (WebCore::headerCallback):
+        (WebCore::ResourceHandleManager::downloadTimerCallback):
+
 2013-06-27  Csaba Osztrogonác  <o...@webkit.org>
 
         Fix cast-align warnings in WebCore/platform/graphics/cairo

Modified: trunk/Source/WebCore/platform/network/curl/CredentialStorageCurl.cpp (152086 => 152087)


--- trunk/Source/WebCore/platform/network/curl/CredentialStorageCurl.cpp	2013-06-27 10:10:08 UTC (rev 152086)
+++ trunk/Source/WebCore/platform/network/curl/CredentialStorageCurl.cpp	2013-06-27 10:28:07 UTC (rev 152087)
@@ -31,7 +31,7 @@
 
 namespace WebCore {
 
-Credential CredentialStorage::getFromPersistentStorage(const ProtectionSpace& protectionSpace)
+Credential CredentialStorage::getFromPersistentStorage(const ProtectionSpace& /* protectionSpace */)
 {
     notImplemented();
     return Credential();

Modified: trunk/Source/WebCore/platform/network/curl/DNSCurl.cpp (152086 => 152087)


--- trunk/Source/WebCore/platform/network/curl/DNSCurl.cpp	2013-06-27 10:10:08 UTC (rev 152086)
+++ trunk/Source/WebCore/platform/network/curl/DNSCurl.cpp	2013-06-27 10:28:07 UTC (rev 152087)
@@ -30,7 +30,7 @@
 
 namespace WebCore {
 
-void prefetchDNS(const String& hostname)
+void prefetchDNS(const String& /* hostname */)
 {
     notImplemented();
 }

Modified: trunk/Source/WebCore/platform/network/curl/ResourceHandleManager.cpp (152086 => 152087)


--- trunk/Source/WebCore/platform/network/curl/ResourceHandleManager.cpp	2013-06-27 10:10:08 UTC (rev 152086)
+++ trunk/Source/WebCore/platform/network/curl/ResourceHandleManager.cpp	2013-06-27 10:28:07 UTC (rev 152087)
@@ -115,13 +115,13 @@
 // libcurl does not implement its own thread synchronization primitives.
 // these two functions provide mutexes for cookies, and for the global DNS
 // cache.
-static void curl_lock_callback(CURL* handle, curl_lock_data data, curl_lock_access access, void* userPtr)
+static void curl_lock_callback(CURL* /* handle */, curl_lock_data data, curl_lock_access /* access */, void* /* userPtr */)
 {
     if (Mutex* mutex = sharedResourceMutex(data))
         mutex->lock();
 }
 
-static void curl_unlock_callback(CURL* handle, curl_lock_data data, void* userPtr)
+static void curl_unlock_callback(CURL* /* handle */, curl_lock_data data, void* /* userPtr */)
 {
     if (Mutex* mutex = sharedResourceMutex(data))
         mutex->unlock();
@@ -312,10 +312,9 @@
      */
     if (header == String("\r\n") || header == String("\n")) {
         CURL* h = d->m_handle;
-        CURLcode err;
 
         long httpCode = 0;
-        err = curl_easy_getinfo(h, CURLINFO_RESPONSE_CODE, &httpCode);
+        curl_easy_getinfo(h, CURLINFO_RESPONSE_CODE, &httpCode);
 
         if (isHttpInfo(httpCode)) {
             // Just return when receiving http info, e.g. HTTP/1.1 100 Continue.
@@ -324,11 +323,11 @@
         }
 
         double contentLength = 0;
-        err = curl_easy_getinfo(h, CURLINFO_CONTENT_LENGTH_DOWNLOAD, &contentLength);
+        curl_easy_getinfo(h, CURLINFO_CONTENT_LENGTH_DOWNLOAD, &contentLength);
         d->m_response.setExpectedContentLength(static_cast<long long int>(contentLength));
 
         const char* hdr;
-        err = curl_easy_getinfo(h, CURLINFO_EFFECTIVE_URL, &hdr);
+        curl_easy_getinfo(h, CURLINFO_EFFECTIVE_URL, &hdr);
         d->m_response.setURL(KURL(ParsedURLString, hdr));
 
         d->m_response.setHTTPStatusCode(httpCode);
@@ -422,7 +421,7 @@
     return sent;
 }
 
-void ResourceHandleManager::downloadTimerCallback(Timer<ResourceHandleManager>* timer)
+void ResourceHandleManager::downloadTimerCallback(Timer<ResourceHandleManager>* /* timer */)
 {
     startScheduledJobs();
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to