[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Build libssh against gcrypt


Andreas Schneider <asn@xxxxxxxxxxxxxx> writes:

> On Wednesday, 2 November 2016 15:51:24 CET Justus Winter wrote:
>> Hello,
>> 
>> a few months ago, I sent a patch series to enable the use of elliptic
>> curves when libssh is built against gcrypt.  As far as I remember I
>> addressed all the issues with it, but the review was put off because of
>> the bad lock thing.  My employer was paid for that work and the customer
>> explicitly requested that the work should be submitted upstream so that
>> they don't have to patch their libssh packages.  Besides,  I hate to see
>> my work bitrot.
>
> Hey,
>
> I would prefer much more smaller patches, but well lets get it in.

Cool.

> I have some 
> patches to squash into yours:
>
> https://git.libssh.org/users/asn/libssh.git/log/?h=master-gcrypt
>
>
> Are you fine if I squash them. They improve readability, debugability and also 
> makes further reviews easier.

Sure.

> There are more things, but I can life with them for now. For the future:
>
> 	int len;
>
> 	len = foo();
> 	if (len == 1)
>
> is easier to debug (gdb$ print len) and to understand then
>
> 	if (foo() == 1)

Wow, that's very verbose given that one can also step into the function,
let it finish and get the result that way in gdb.  But it is a matter of
taste of course.

Cheers,
Justus

Attachment: signature.asc
Description: PGP signature


Follow-Ups:
Re: Build libssh against gcryptAndreas Schneider <asn@xxxxxxxxxxxxxx>
References:
Build libssh against gcryptJustus Winter <justus@xxxxxxxxxxx>
Re: Build libssh against gcryptAndreas Schneider <asn@xxxxxxxxxxxxxx>
Archive administrator: postmaster@lists.cynapses.org