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

Small issue with it thinking that the file "4.2BSD-fortunes" is a percentage. #94

Closed
wants to merge 2 commits into from
Closed

Small issue with it thinking that the file "4.2BSD-fortunes" is a percentage. #94

wants to merge 2 commits into from

Commits on May 7, 2024

  1. Fixed a bug with it not being able to read the file "4.2BSD-fortunes"… 

    … since it had a \'.\' in it. The way I handled it is too loop through the entire string after the '.' and check for anything that isn't a digit. If so it will know it is not an attempt to use a non-integer percent value. I think a smarter way to handle this is to check if there is a \'%\' at the end of the string
    257m committed May 7, 2024
    Configuration menu
    Copy the full SHA
    b355508 View commit details
    Browse the repository at this point in the history

  2. Ok I redid it so we simply check for a \'%\' at the end of string ins… 

    …tead of the older naive approach. This should work fine.... Hopefully
    257m committed May 7, 2024
    Configuration menu
    Copy the full SHA
    2ff7c66 View commit details
    Browse the repository at this point in the history