search

resurrecting-open-source-projects / scrot

SCReenshOT - command line screen capture utility
Other
495 stars 49 forks source link

Pick one of `int` or `bool` and stick to it #260

Closed guijan closed 1 year ago

guijan commented 1 year ago

Scrot uses a mix of the int and bool types to do boolean logic: https://github.com/resurrecting-open-source-projects/scrot/blob/c0f029e6150a743747dbdc7105d1cc4e98260b15/src/options.c#L440-L454 https://github.com/resurrecting-open-source-projects/scrot/blob/c0f029e6150a743747dbdc7105d1cc4e98260b15/src/options.c#L154-L157

Contributors should decide which one (and only one) should be used across the source code for consistency.