Do not call SetErrorForJob before the job is done

The code in VolumeDataRequestProcessor::WaitForCompletion would call SetErrorForJob also in the case where it reached the timeout and the job wasn't actually done, and this could lead to at deadlock. It's actually not necessary to set the error here at all, it will also be set in IsCanceled when you try to get the error.

Fixes #197 (closed)

src/OpenVDS/VDS/VolumeDataRequestProcessor.cpp

@@ -2032,10 +2032,6 @@ bool VolumeDataRequestProcessor::WaitForCompletion(int64_t jobID, int millisecon
     m_jobs.erase(job_it);
     return true;
   }
-  if (job->cancelled)
-  {
-    SetErrorForJob(job);
-  }
   return false;
 }