On 23 August 2012 12:49, Bert Freudenberg <bert at freudenbergs.de> wrote:
> On 2012-08-23, at 11:24, Frank Shearar wrote:
>
>> It looks like PackageInfo possibly isn't catching the AddEvent sent by
>> ClassDescription >> #addSelector:withMethod:notifying: but I'm just
>> guessing: I don't know enough about this part of the system.
>
>
> No, PackageInfo is not event-based. It does not "store" classes or methods. Instead, when you ask it for classes and methods in the package, it looks at the system.
>
> What must be happening is that the class's organizer gets out-of-sync with the method dictionary. The System Browser only looks at the organization to display methods. So if a selector is not in the organizer, the Browser does not show it at all.

OK. So the organizer loses the reference because ObjectAsMethodWrapper
calls #removeSelectorSilently:, which eventually calls
ClassDescription >> #removeSelector:. It re-adds the now-unwrapped
CompiledMethod to the method dictionary with #addSelector:withMethod:
which eventually calls ClassDescription >>
#basicAddSelector:withMethod:. However, this does _not_ add the
selector to the organization's elementArray, and Bad Things Happen.

I'm trying to find how things get _into_ a Categorizer's elementArray.
I suspect it's via ClassDescription >>
#addAndClassifySelector:withMethod:inProtocol:notifying:. A Browser
would do this via accepting code, while if you're hacking on the
method dictionary directly, you'll need to do this yourself.

This looks like it would do the trick. At least, it makes the failing test pass:

ObjectAsMethodWrapper >> uninstall
	| category |
	oldMethod isNil ifTrue: [^self].
	category := wrappedClass organization categoryOfElement: selector.
"<-- store the original category"
	wrappedClass removeSelectorSilently: selector.
	wrappedClass addSelector: selector withMethod: oldMethod.
	wrappedClass organization classify: selector under: category. "<--
categorise the restored method."

frank

> This is different in Monticello - if you click "browse" in MC it makes a package snapshot by asking PackageInfo, which iterates through the method dictionary. After running the test this blows up, because it finds the "bar" method, but that is not in the organizer, so its category is nil and you get a walkback.
>
> Events are only used to mark the Monticello working copy dirty. Not to actually keep track of method additions.
>
> - Bert -
>
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ObjectAsMethodWrapper-fbs.19.mcz
Type: application/octet-stream
Size: 6570 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20120823/e1230655/ObjectAsMethodWrapper-fbs.19.obj