2015-08-24 13:25 GMT+02:00 Thierry Goubier <[email protected]>:
> 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 <[email protected]>: > >> 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 >> > >
