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

added new function: get_full_functions #1

Merged
merged 2 commits into from
Feb 16, 2013
Merged

added new function: get_full_functions #1

merged 2 commits into from
Feb 16, 2013

Conversation

bayashi
Copy link
Contributor

@bayashi bayashi commented Feb 14, 2013

欲しかったので足してみましたー
PODはいじってないす。

@tokuhirom
Copy link
Owner

I think "get_all_functions" is even better name.

@tokuhirom
Copy link
Owner

And, document is required for patch.

@bayashi
Copy link
Contributor Author

bayashi commented Feb 15, 2013

exactly! I should use keys %{"${klass}::"}.
I named "get_full_function" reason why It was followed Class::Inspector.
Anyway, this PR makes no sense.
thank you.

@bayashi bayashi closed this Feb 15, 2013
@bayashi bayashi reopened this Feb 15, 2013
added POD about `get_full_functions` and tiny sorted POD
tokuhirom added a commit that referenced this pull request Feb 16, 2013
added new function: get_full_functions
@tokuhirom tokuhirom merged commit 91612af into tokuhirom:master Feb 16, 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.

2 participants