Angus Leeming <[EMAIL PROTECTED]> writes:

| On Monday 29 July 2002 10:56 am, Lars Gullik Bj�nnes wrote:
>> Angus Leeming <[EMAIL PROTECTED]> writes:
>> | Lars,
>> |
>> | have you any ideas about how to fix this set of warnings in the new boost
>> | code?
>>
>> does cxx support "long long" or not?
>
| Yes, it supports it and I can easily turn the warning off. However, the 
| compiler is telling me that "long long" is not part of the standard and that 
| I should not, therefore, be using it. So, I might rephrase my original 
| question as "Why the hell is boost using it?"

Because it is a _very_ common extension, and almost _all_ compilers
support it.

>> It is do support it, than I cannoth fathom why it complains... do yo
>> have a switch turned on that you shouldn't?
>>
>> | cxx: Warning: ../../../devel/boost/boost/signals/connection.hpp, line
>> | 154: #767-D
>> |           conversion from pointer to smaller integer
>> |       assert(con.get());
>> | ------^
>>
>> So this is what your "!= 0" was about.
>>
>> Is my guess that
>>
>>    sizeof(void*) == sizeof(long) > sizeof(int)
>>
>> correct?
>
| Exactly so. 
>
| sizeof(int)    = 4
| sizeof(long)   = 8
| sizeof(void *) = 8
>
| Actually, I hadn't realised that this warning was in among those long long 
| ones. This happens in three places in the boost code, IIRC. I'll create the 
| patch again and send it to you. Ok? 

I want to run it by the boost folks as well, this must be a problem
that pops up on all 64bit platforms.

| Angus
>
| ps I submitted the patch to boost, but no-one replied to tell me if it went 
| in or not. Perhaps you could check boost cvs...

It did not go in.

-- 
        Lgb

Reply via email to