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

Fix panic when columns < NO_LANG_HEADER_ROW_LEN #331

Merged
merged 1 commit into from May 6, 2019

Conversation

Projects
None yet
2 participants
@ocisly
Copy link
Contributor

commented May 6, 2019

Thank you for making Tokei! This fixes the following panic by ensuring that columns is always wide enough to print the header row:

$ cargo run -- --columns=66
thread 'main' panicked at 'attempt to subtract with overflow',
src/cli_utils.rs:64:17
Fix panic when columns < NO_LANG_HEADER_ROW_LEN
    $ cargo run -- --columns=66
    thread 'main' panicked at 'attempt to subtract with overflow',
    src/cli_utils.rs:64:17
@XAMPPRocky

This comment has been minimized.

Copy link
Owner

commented May 6, 2019

Thank you for your pull request! Sorry the CI isn't working right now. This part of code isn't tested anyway 😅 So I'm happy to merge it in. Congrats on your first contribution! 🎉

@XAMPPRocky XAMPPRocky merged commit c2a1665 into XAMPPRocky:master May 6, 2019

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.