curl / Mailing Lists / curl-library / Single Mail

curl-library

Re: Tests 1060, 1061 fail

From: Spork Schivago <sporkschivago_at_gmail.com>
Date: Mon, 9 Jan 2017 20:35:06 -0500

Daniel,

I think that's probably one of the most important things about programming.
  Making the code as easy to read and understand as possible. I tried
coming up with a solution using a few different styles of loops, to see if
I could do away with your goto (after I sent in my first message) but I
couldn't find away to do it that was nice either. Granted, I'm not nearly
as good as you guys at writing code and I haven't really written serious
code in years.

Anyway, thanks for sharing!

On Mon, Jan 9, 2017 at 5:44 PM, Daniel Stenberg <daniel_at_haxx.se> wrote:

> On Mon, 9 Jan 2017, Spork Schivago wrote:
>
> That's what I thought you were doing originally. Using the goto just as a
>> simple fix, to see if it'd correct the problem. But then I thought I saw a
>> post where you were going to commit it to the main branch or something
>> along those lines.
>>
>
> In the end I went with the goto solution anyway and I've now pushed that
> to git. Changing it to a while loop turned out to end up with a lot more
> code that certainly was not easier to follow or understand than the goto
> approach...
>
>
> --
>
> / daniel.haxx.se
> -------------------------------------------------------------------
> List admin: https://cool.haxx.se/list/listinfo/curl-library
> Etiquette: https://curl.haxx.se/mail/etiquette.html
>

-------------------------------------------------------------------
List admin: https://cool.haxx.se/list/listinfo/curl-library
Etiquette: https://curl.haxx.se/mail/etiquette.html
Received on 2017-01-10