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

Track and remove unused strings #1

Open
AntoineTurmel opened this issue Jun 18, 2012 · 7 comments
Open

Track and remove unused strings #1

AntoineTurmel opened this issue Jun 18, 2012 · 7 comments

Comments

@AntoineTurmel
Copy link
Member

All that are only on .dtd/.properties but not used anymore in the code.

@saebekassebil did a node.js tool for this (dtd only) : https://gist.github.com/2882364

@AntoineTurmel
Copy link
Member Author

We also have to be sure to keep possible features strings (Podcasts, Equalizer presets...)

@AntoineTurmel
Copy link
Member Author

@saebekassebil

Is it possible for you to update the script to also parse for strings in .properties (JavaScript files) and also check for strings used on default add-ons ? Thanks :)

@saebekassebil
Copy link

@GeekShadow I'd love to, but I've got summer vacation and I won't be able to code in about 6 weeks starting from tomorrow! Hope I'll be able to lend a hand when I get back :)

@AntoineTurmel
Copy link
Member Author

@saebekassebil great :) Enjoy your holidays !

@AntoineTurmel
Copy link
Member Author

Improved a bit the search.js nodejs tool to parse xml files and to parse directly from top folder (to catch all other files such as feathers and add-ons)

https://gist.github.com/3026206/

@AntoineTurmel
Copy link
Member Author

First review :
prefixed equalizer -> keep them
prefixed jumpto -> do we keep them ?
prefixed umsd -> no clues where it's used.

@AntoineTurmel
Copy link
Member Author

@saebekassebil
Are you still interested into updating you script to support parsing .properties file too ? It would be nice 😃

Your script should allow parsing .js, .jsm and .cpp files too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants