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.

Sign up for GitHub

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

Jump to bottom

Feature request: detect exported identifiers with no defined methods #9

Open
jiahao opened this issue Jul 18, 2014 · 1 comment
Open

Feature request: detect exported identifiers with no defined methods #9

jiahao opened this issue Jul 18, 2014 · 1 comment

Comments

@jiahao
Copy link

jiahao commented Jul 18, 2014

I've been going through Base and Base.LinAlg and removed a couple of obsolete functions that were still being exported. TypeCheck.check_return_types has been really useful since it throws an error for each such function; for example:

ERROR: cholpfact not defined
 in check_all_module at /Users/jiahao/.julia/v0.3/TypeCheck/src/TypeCheck.jl:13
 in check_return_types at /Users/jiahao/.julia/v0.3/TypeCheck/src/TypeCheck.jl:44
 in process_options at ./client.jl:213
 in _start at ./client.jl:354

It would be nice, however, to collect all such identifiers and report them all together.
jiahao added a commit to JuliaLang/julia that referenced this issue Jul 18, 2014
@jiahao
Remove deprecated exports from Base and Base.LinAlg
249078d 
- A_mul_B
- Ac_mul_b_RFP
- qrp
- cholpfact, cholpfact!
- qrfact, qrfact!
- LinAlg.LUTridiagonal
- LinAlg.solve

Previously, callling any of these resulted in an `ERROR: ____ not defined`.

HT astrieanna/TypeCheck.jl#9
@vtjnash
Copy link
Contributor

vtjnash commented Jul 21, 2014

my unimaginatively named "various updates" pull request includes this feature: vtjnash@ef5daf7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jiahao @vtjnash