Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix on requirement #4

Closed
wants to merge 52 commits into from
Closed

fix on requirement #4

wants to merge 52 commits into from

Conversation

RemiDufloer
Copy link
Owner

No description provided.

RemiDufloer and others added 30 commits February 21, 2024 12:04
ProtoObject does not define hash method, the proxy therefore was forwarding the hash message to the wrapped object.
But the wrapped object and the proxy have swapped their references.
Therefore the pointers to the objects are now pointing to the proxy.
And the hash of the object is the hash of the proxy now and vice versa.

So the proxy must understand the hash method in order to maintain the consistency of the indexed collections.
Some messages from ProtoObject must be forwarded to the meta-object.
So nil must be the superclass of the Proxy and the proxy must define the identity related methods.
The identity related methods are copied from ProtoObject.
Fix proxy not working when using a ProtoObject as superclass
…f test For mop.

Features : Switch the responsabillity to install link on Instrumentation
…an instance of NoRequirement) for the application. So BlockOperation is not alone with requirement so i decide to move thios on super class
…, and check if instrumentation need to install.
…now if the ast node is a variable node

	add Test and clean code
…you have the possibilities to create an instrumentation with requirement step by step
@RemiDufloer RemiDufloer closed this Apr 4, 2024
@RemiDufloer RemiDufloer deleted the test-demo branch April 4, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants