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

Move callee_id out of ast::expr #6885

Merged
merged 2 commits into from
Jun 2, 2013
Merged

Conversation

erickt
Copy link
Contributor

@erickt erickt commented Jun 1, 2013

The callee_id in ast::expr in only used in a couple expression variants. This moves the callee_id into those branches to make it more clear when its should be used.

Also, it fixes a bug in a std::run test when there is a symlink in the path rust where was checked out.

@catamorphism
Copy link
Contributor

Needs rebase. Once you've rebased, I'll review this, because I originally made the choice to put callee_id in the expr. I think it was just because I was lazy, but I'll see how the code looks and maybe I'll remember if there was another reason.

@erickt
Copy link
Contributor Author

erickt commented Jun 1, 2013

@catamorphism: Just pushed a rebase.

@erickt
Copy link
Contributor Author

erickt commented Jun 1, 2013

@catamorphism and another one. Added the requested fixme too: #6890.

bors added a commit that referenced this pull request Jun 2, 2013
The `callee_id` in `ast::expr` in only used in a couple expression variants. This moves the `callee_id` into those branches to make it more clear when its should be used.

Also, it fixes a bug in a std::run test when there is a symlink in the path rust where was checked out.
@bors bors closed this Jun 2, 2013
@bors bors merged commit 23808ef into rust-lang:incoming Jun 2, 2013
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.

3 participants