Fix check for empty names #1

Merged

linus merged 1 commit from archbtw/skitmacka:empty-names-fix into main

2024-09-04 08:57:43 +00:00

archbtw commented

2024-09-04 08:47:53 +00:00

Collaborator

Även om -n-flaggan är tom kommer strings.Split("", ",") att producera en slice som innehåller en tom sträng snarare än en nolllängd-slice, så att len(names) alltid kommer att vara minst 1 och därmed ge följande:

$ ./skitmacka

    ________
 (´\        ´\
  ´-\ GRATTIS \      ... !
     \   (@)   \     skitmackan är din.
     _\   |\    \
    ( _)_________)
     ´----------´

Denna PR kollar i stället efter en tom sträng direkt i names-flaggan och skriver i så fall ut usage och bailar ur.

Även om `-n`-flaggan är tom kommer `strings.Split("", ",")` att producera en slice som innehåller en tom sträng snarare än en nolllängd-slice, så att `len(names)` alltid kommer att vara minst 1 och därmed ge följande: ``` $ ./skitmacka ________ (´\ ´\ ´-\ GRATTIS \ ... ! \ (@) \ skitmackan är din. _\ |\ \ ( _)_________) ´----------´ ``` Denna PR kollar i stället efter en tom sträng direkt i names-flaggan och skriver i så fall ut usage och bailar ur.

archbtw added 1 commit

2024-09-04 08:47:54 +00:00