lightyear icon indicating copy to clipboard operation
lightyear copied to clipboard

space parses zero or more whitespace, while it would make more sense to parse only one!

Open bgaster opened this issue 9 years ago • 0 comments

Currently Lightyear.Strings.space parses zero or more whitespace, which seems strange given it singular naming. Moreover, this is in contrast to Parsec which defines:

space

and

spaces

to separate these two cases. From Parsec.Text.Chars:

-- | Skips /zero/ or more white space characters. See also 'skipMany'. spaces :: (Stream s m Char) => ParsecT s u m () spaces = skipMany space <?> "white space"

-- | Parses a white space character (any character which satisfies 'isSpace') -- Returns the parsed character.

space :: (Stream s m Char) => ParsecT s u m Char space = satisfy isSpace <?> "space"

bgaster avatar Sep 25 '15 09:09 bgaster