2015-08-24 13:25 GMT+02:00 Thierry Goubier <thierry.goubier@gmail.com>:

I'm trying on 50264, and, yes, there is something strange:

Monticello diff only picks up two of the changes, instead of three.

Ok, by reformatting, ended up with exactly the same code for one of them :(

Saved in the package cache and I have everything saved.

Some suspicious/wrong data in your image?

Thierry

��

Thierry

2015-08-24 12:05 GMT+02:00 Marcus Denker <marcus.denker@inria.fr>:
Hi,

I have a *very* strange bug.

I am trying to add change these three methods in class ��TemporaryVariable��:
(I copy the new version):

ensureProperties
�� method saveTemp: self.
�� ^ Properties at: self ifAbsentPut: WeakKeyDictionary new

removePropertiesIfEmpty
�� ^ Properties at: self ifPresent: [ :dict |
�� dict ifEmpty: [
�� method removeSavedTemp: self.
�� Properties removeKey: self ] ]

hash

�� ^ (name hash
�� bitXor: method hash)
�� bitXor: (startpc ifNil: 0)

Now when I save the MC package, these changes are *not* saved, even though when I diff against the repository
of Pharo5, they are shown as different.

The only special things I did is to move TemporaryVariable to a new tag some updates ago. And I had extension
methods in *slot where the same thing happened, but that was fixed by putting them into *Slot.

This looks like some wired RPackage problem. But the RPackage seems to correctly know TemporaryVariable and these three selectors
as local from the package Slot.

I really have no idea what the problem could be.

�� Marcus