cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: Gopher patches for cURL (includes test suite)

From: Cameron Kaiser <spectre_at_floodgap.com>
Date: Wed, 25 Aug 2010 08:24:02 -0700 (PDT)

> > Okay, here we go. This is gopher for cURL round 2:
>
> Thanks a lot for your contribution! I've just merged your patches with these
> additional changes by me:
>
> * converted the tests to instead use the HTTP server (with a --gopher
> switch) to make them work better
>
> * prevented a busy-loop
>
> * fixed a memory leak
>
> There's one (minor) problem left in the gopher code and I've added a big
> comment blob in gopher.c about it:
>
> The gopher_do() function sends the request in a loop-style which causes the
> function to block until done. This is not fine for when the code is used by
> the multi interface. Instead it should then send as much as possible in the
> first shot and then continue sending the rest in a "doing" function.

Noted. I'll watch for this and I'll be happy to rewrite it if this turns
out to be a problem. Thanks for the accept and the review!

-- 
------------------------------------ personal: http://www.cameronkaiser.com/ --
  Cameron Kaiser * Floodgap Systems * www.floodgap.com * ckaiser_at_floodgap.com
-- You do not have a soul. You are a soul. You have a body. -- C.S. Lewis -----
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2010-08-25