Subject: Re: [libssh2] [libssh2-cvs] jehousley: libssh2/include libssh2.h, 1.76, 1.77

Re: [libssh2] [libssh2-cvs] jehousley: libssh2/include libssh2.h, 1.76, 1.77

From: James Housley <jim_at_thehousleys.net>
Date: Sat, 14 Jul 2007 17:12:12 -0400

On Jul 14, 2007, at 4:04 PM, Daniel Stenberg wrote:

> On Sat, 14 Jul 2007, CVS commit messages for libssh2 wrote:
>
>> For now make LIBSSH2_APINO a long, some compilers make constants
>> an int it
>> seems.
>
> [...]
>
>> -#define LIBSSH2_APINO 200706151200
>> +#define LIBSSH2_APINO 200706151200L
>
> On current 32 bit systems, ints and longs are the same size: 32 bit.
> 200706151200 is a value that doesn't fit in 32 bits, be it int or
> long.
>
> 2^32 is 4294967296, and this APINO define is almost 47 times larger...

You are right, it would need to be a LONG LONG, which is not fully
portable. And the commit it libcurl might quite the preprocessor,
but doesn't really work either then.

Jim

--
/"\   ASCII Ribbon Campaign  .
\ / - NO HTML/RTF in e-mail  .
  X  - NO Word docs in e-mail .
/ \ -----------------------------------------------------------------
                      http://www.FreeBSD.org     The Power to Serve
jim@TheHousleys.Net  http://www.TheHousleys.net
---------------------------------------------------------------------
The wise man built his network upon Un*x.
     The foolish man built his network upon Windows.
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
libssh2-devel mailing list
libssh2-devel_at_lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libssh2-devel
Received on 2007-07-14