cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: libssh2 patch 10/17

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Thu, 19 Oct 2006 22:34:16 +0200 (CEST)

On Thu, 19 Oct 2006, James Housley wrote:

> Are you looking at the current patch? Because scp now does that, and sftp
> doesn't work and hasn't be edited.

Can you please post it again? I'm looking at the code now again and I still
see the same things I wrote about. Did you possibly post the wrong package?

>> 2 - You use very long source lines. We use less than 80 columns per line in
>> this project.
>
> I have mine set to 132, I will try and keep them down to 80. I will
> probably go back and edit/trim at some point.

That's one of the things I require code to adhere to before I commit.

-- 
  Commercial curl and libcurl Technical Support: http://haxx.se/curl.html
Received on 2006-10-19