I have to concur, something very strange is wrong. #50265
Load Zinc-Tests-SvenVanCaekenberghe.231 from http://mc.stfx.eu/ZincHTTPComponents ZnEntityTests>>#testUnspecifiedEncoding should have today as latest version and it simply does not (the code is wrong too), I can't imagine how that is possible. And this is with loading code ... seems quite dangerous. > On 24 Aug 2015, at 13:30, Thierry Goubier <[email protected]> wrote: > > > > 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 > >
