[Vm-dev] other 64bit changes, header files, and smalltalk

Eliot Miranda eliot.miranda at gmail.com
Tue Jan 12 19:57:30 UTC 2010


On Tue, Jan 12, 2010 at 11:52 AM, John M McIntosh <
johnmci at smalltalkconsulting.com> wrote:

>
> Mmm no, the history of the squeakFileOffsetType was originally for the use
> of file system 64bit points back when 64bit support Windows was a
> footnote in some Microsoft engineers diary.  It was first introduced on the
> mac version of the VM with other variations following years? later.
>
> Technically it is a platform specific entity since not all file systems
> support 64bit values, so it's improper to consider for other usages outside
> the file system support plugins.
>

OK, but in any case IMO the usqLong/sqLong defines belong alongside the
sqInt/usqInt ones.


>
>
> On 2010-01-12, at 11:42 AM, Eliot Miranda wrote:
>
>
>
> On Tue, Jan 12, 2010 at 7:54 AM, Andreas Raab <andreas.raab at gmx.de> wrote:
>
>>
>> Igor Stasenko wrote:
>>
>>> Btw,  about that function. Its using a squeakFileOffsetType, which is
>>> platform specific, and i had hard times
>>> trying to deal with right header inclusion order imposing dependency
>>> of interpreter from platform code, which, IMO should be avoided.
>>> I propose to change it to
>>> typedef  unsigned long long vmFileOffsetType;
>>> and use this type instead.
>>>
>>
>> I'm not sure if MSVC supports long long nowadays. It didn't used to - it
>> used to require __int64 which is why the definition of squeakFileOffsetType
>> is external.
>>
>
> We should just use usqLong.  The platform-specific definition in
> platforms/Cross/vm/sqVirtualMachine.h can be moved to sqMemoryAccess.h so
> that usqLong & sqLong are also available once sqInt & usqInt are.
>
>
>
>>
>> Cheers,
>>  - Andreas
>>
>>
>
> --
> ===========================================================================
> John M. McIntosh <johnmci at smalltalkconsulting.com>   Twitter:
>  squeaker68882
> Corporate Smalltalk Consulting Ltd.  http://www.smalltalkconsulting.com
> ===========================================================================
>
>
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20100112/5f573c59/attachment.htm


More information about the Vm-dev mailing list