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

Add @usingif #2444

Closed
wants to merge 1 commit into from
Closed

Add @usingif #2444

wants to merge 1 commit into from

Conversation

timholy
Copy link
Sponsor Member

@timholy timholy commented Mar 2, 2013

Useful? This might help with #2025.

@pao
Copy link
Member

pao commented Mar 2, 2013

I think this makes more sense in Pkg than in Meta; it's not really a metaprogramming tool.

@mlubin
Copy link
Member

mlubin commented Apr 8, 2013

+1 bump

@quinnj
Copy link
Member

quinnj commented Jun 13, 2013

I ran into needing this the other day. +1

@timholy
Copy link
Sponsor Member Author

timholy commented Jun 14, 2013

Update: I think the "cleaner way" was wrong, so I don't currently have a cleaner approach.

I've since nuked the branch this was based on, but I can clean it up and push if this is something mergeable. The suggestions I've gotten:

  • Make the status an optional second argument, rather than the first argument. I suspect you'd almost always want to know whether the module was loaded, so I am a bit skeptical this would be used much. Unless there's a better way to test whether it's been loaded? names is not exactly the most efficient approach, but we could go that way.
  • I can put this into Pkg rather than Meta

If I don't hear objections in the next week or so, I'll make these changes and just push it.

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

4 participants