Skip to content

Remove all attributes/ display information work, delegating it to new project: WLA #48

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

Open
sts10 opened this issue Aug 31, 2023 · 0 comments

Comments

@sts10
Copy link
Owner

sts10 commented Aug 31, 2023

Tidy's getting a bit large (4k lines!). So I'm thinking of moving all of the word list information/auditing over to a new project that I'm tentatively calling WLA.

My question: Should I now remove all this list attributes/information code from Tidy? Tidy users, thanks to the magic of Unix philosophy/piping, can simply pipe Tidy's output over to wla, which will perform the same as running tidy -AAAA:

tidy -D t eff.txt | wla

We could remove not only all the attribute printing code, but also the -G/-g options, that I bet are potentially confusing for users when comparing them to the -D/-d options!

Besides reducing codebase size, this also allows WLA to act more like a "true" auditor of word lists. One issue with having Tidy serve as both a word list creator and a word list auditing tool is that, if a list had duplicate or blank lines, Tidy would quietly remove these before printing attributes, which is kind of a lie. For example:

blue
green
red

blue

tidy -A counts this as 3 words, rather 4 or 5, since it automatically removes duplicates and blank lines before calculating attribute values, like list length.

In contrast, we can have wla count this as 5 "lines", then warn users that there are both blank and duplicate lines present.

@sts10 sts10 changed the title Remove all attributes work, delgating it to new project: WLA Remove all attributes/ display information work, delegating it to new project: WLA Oct 28, 2023
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

No branches or pull requests

1 participant