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

Remove unused code/functions in base/R #244

Closed
ivokwee opened this issue Mar 17, 2023 · 2 comments
Closed

Remove unused code/functions in base/R #244

ivokwee opened this issue Mar 17, 2023 · 2 comments
Assignees
@ncullen93 @ivokwee

Comments

@ivokwee
Copy link
Member

ivokwee commented Mar 17, 2023

Would be good to remove unused functions in base/R and elsewhere. The idea would be to scan each file for function declarations and function names beings used and make a dependency graph so we can isolate function that are not being used. I believe there is much dead code...
@ivokwee
Copy link
Member Author

ivokwee commented Mar 20, 2023

I have started making such a tool in tools/utils/code-analyzer.R. . It shows where a function is defined, where it is used, how many time defined/used. Not perfect yet but looks useful.

image

@ncullen93
Copy link
Contributor

Closing because base/R doesn't exist anymore. Unnecessary code should not be removed from playbase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ncullen93 ncullen93

@ivokwee ivokwee

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ncullen93 @ivokwee