VM assert bugfixes/suspicious stuff

Brian T. Rice water at tunes.org
Tue Jul 27 10:49:07 PDT 2004


Applied. Thanks!

Lee, if you need to make changes to this, please inform the list and let 
people know why the changes need to be made, so that contributors 
understand what the design parameters are. If we don't inform them, I 
would not blame them for not contributing.

Lendvai Attila wrote:
> hi!
> 
> here are some changes to make ASSERT work and to eliminate a few
> warnings. please take a look at the diff file before applying as it may
> contain suspicious places. like unused local variables, especiallt the
> shiftAmountInWords.
> 
> 
> please take a look at a crash captured while debugging. it's the
> included png, objectSize is 0.
> 
> 
> novo, take a look at these warnings:
> 
> unary minus on unsigned, result is still unsigned:
> 
> 	- PSObjectHeap_adjustAllOopsBy_ (CurrentMemory, - (unsigned)
> CurrentMemory -> memory);
> 
> 
> 
>>= signed/unsigned mismatch:
> 
> 
> 	- if (!(((((map->slotTable)->slots)[slot]).name ==
> CurrentMemory->NilObject) ||
> (ObjectPointer_asSmallInt((((map->slotTable)->slots)[slot]).offset) >=
> delegateOffset)))
> 
> 
> hope it helps,
> 
> - attila
>  
> PS: I don't cc the list due to the included files...
> 
> 
> ------------------------------------------------------------------------
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: water.vcf
Type: text/x-vcard
Size: 208 bytes
Desc: not available
Url : /archives/slate/attachments/20040727/2357f7ae/water.vcf


More information about the Slate mailing list