rename easy.{ready => done}

Also eliminate try/catch around `wait(easy.done)` since it can't throw.

Author: StefanKarpinski

src/Curl/Easy.jl

@@ -1,7 +1,7 @@
 mutable struct Easy
     handle   :: Ptr{Cvoid}
     input    :: IO
-    ready    :: Threads.Event
+    done     :: Threads.Event
     seeker   :: Union{Function,Nothing}
     output   :: IO
     progress :: Function

src/Curl/Multi.jl

@@ -85,7 +85,7 @@ function check_multi_info(multi::Multi)
             easy = unsafe_pointer_to_objref(easy_p_ref[])::Easy
             @assert easy_handle == easy.handle
             easy.code = message.code
-            notify(easy.ready)
+            notify(easy.done)
         else
             @async @error("curl_multi_info_read: unknown message", message)
         end

src/Downloads.jl

@@ -343,10 +343,8 @@ function request(
 
                 # do the request
                 add_handle(downloader.multi, easy)
-                try wait(easy.ready) # can this throw?
-                finally
-                    remove_handle(downloader.multi, easy)
-                end
+                wait(easy.done)
+                remove_handle(downloader.multi, easy)
 
                 # return the response or throw an error
                 response = Response(get_response_info(easy)...)