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

Zsuper mutator #73

Merged
merged 2 commits into from
Jul 23, 2013
Merged

Zsuper mutator #73

merged 2 commits into from
Jul 23, 2013

Conversation

dkubb
Copy link
Collaborator

@dkubb dkubb commented Jul 23, 2013

This branch mutates zsuper nodes.

@dkubb dkubb closed this Jul 23, 2013
@dkubb dkubb reopened this Jul 23, 2013
@dkubb
Copy link
Collaborator Author

dkubb commented Jul 23, 2013

@mbj this causes one new mutation to be found in axiom-types, eg:

evil:Axiom::Types::Object.infer:/Users/dkubb/Dropbox/Home/Programming/open-source/axiom-types/lib/axiom/types/object.rb:25:4666a
@@ -1,4 +1,4 @@
 def self.infer(object)
-  ((super) || (infer_from_primitive_class(object)))
+  ((nil) || (infer_from_primitive_class(object)))
 end
(08/09)  88% - 1.11s

The part that I don't understand is why statement deletion didn't catch this in the first place? You'd think if it changed that to just infer_from_primitive_class(object) that it would have the same effect.

@mbj
Copy link
Owner

mbj commented Jul 23, 2013

@dkubb Because statement deletion only works with begin and kwbegin nodes.

mbj added a commit that referenced this pull request Jul 23, 2013
@mbj mbj merged commit b87b33f into master Jul 23, 2013
@dkubb dkubb deleted the zsuper-mutator branch July 23, 2013 14:02
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.

None yet

2 participants