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

File office2any.sh Optimized via AWK #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

File office2any.sh Optimized via AWK #2

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 15, 2019

The commit message has the main information about the changes. However, I forgot to mention that I also made the exiting a touch cleaner, as it now happens from help() itself, based on the integer (first position) provided to the function.

terminalforlife added 2 commits November 15, 2019 12:19
Now `awk` can take of help almost entirely on its own, saving the need
for `sed`, but I've also added `-h|--help` to `help()` with description
text shown in `ls --help`, since it looks as though you were going for
a traditional approach here.

I've also made `awk` use `-P` to adhere to POSIX, in-case it's desired.

As long as the formatting is the same as is already present in the case
statement, there shouldn't be any issues with additional file formats.
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.

0 participants