Skip to content
This repository
Browse code

Revert "changed: No need to get FTP filetimes for Exists() calls"

This reverts commit e373cd5.
  • Loading branch information...
commit 8332e8b0355ba35b034433b78b159a6829ee7acd 1 parent 1145deb
authored February 11, 2013

Showing 1 changed file with 4 additions and 1 deletion. Show diff stats Hide diff stats

  1. 5  xbmc/filesystem/CurlFile.cpp
5  xbmc/filesystem/CurlFile.cpp
@@ -1084,8 +1084,11 @@ bool CCurlFile::Exists(const CURL& url)
1084 1084
   g_curlInterface.easy_setopt(m_state->m_easyHandle, CURLOPT_WRITEDATA, NULL); /* will cause write failure*/
1085 1085
 
1086 1086
   if(url2.GetProtocol() == "ftp")
  1087
+  {
  1088
+    g_curlInterface.easy_setopt(m_state->m_easyHandle, CURLOPT_FILETIME, 1);
1087 1089
     g_curlInterface.easy_setopt(m_state->m_easyHandle, CURLOPT_FTP_FILEMETHOD, CURLFTPMETHOD_NOCWD);
1088  
-  
  1090
+  }
  1091
+
1089 1092
   CURLcode result = g_curlInterface.easy_perform(m_state->m_easyHandle);
1090 1093
   g_curlInterface.easy_release(&m_state->m_easyHandle, NULL);
1091 1094
 

2 notes on commit 8332e8b

Memphiz
Owner

Would be nice to have a comment on each revert - why it is actually reverted ...

arnova
Collaborator

Because it broke Exists() for FTP. Apparently CURLOPT_FILETIME is not only needed for :Stat() but for proper existance checking as well.

Please sign in to comment.
Something went wrong with that request. Please try again.