diff --git a/CHANGELOG.md b/CHANGELOG.md index dbacf18..cf21bca 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,10 +2,10 @@ ## v1.3.5 - 2024-11-19 -TODO - -- Fixed bug with parsing base prefix strings later in the string -- Fixed bug with base 0 and no prefix would still parse +- Fixed a bug where base prefix substrings would be recognized as such later in the string. + - The `0b` in `0XDEAD_BEEF0b` should be catorgized as digits and not a base prefix. +- Fixed a bug where, in base 0 mode, `lenient_parse` would parse a string with an incomplete base prefix. + - `01` should reject, as it is missing a specifier (`b`, `B`, `o`, `O`, `x`, `X`). ## v1.3.4 - 2024-11-19 diff --git a/src/lenient_parse/internal/parser.gleam b/src/lenient_parse/internal/parser.gleam index 0e4b09c..f3bba83 100644 --- a/src/lenient_parse/internal/parser.gleam +++ b/src/lenient_parse/internal/parser.gleam @@ -231,9 +231,6 @@ fn parse_sign( } } -// TODO: Clean up the index range vs start and end index stuff here -// TODO: Delete {}s in case -// TODO: Clean up new_rest duplication fn parse_base_prefix( tokens tokens: List(Token), index index: Int, diff --git a/test/tokenizer_test.gleam b/test/tokenizer_test.gleam index 0142b35..1be4c4d 100644 --- a/test/tokenizer_test.gleam +++ b/test/tokenizer_test.gleam @@ -437,5 +437,3 @@ pub fn tokenize_int_with_all_whitespace_characters_test() { |> tokenizer.tokenize_int |> expect.to_equal(expected_tokens) } -// TODO: all tokenizer tests and see what isn't needed now that base prefix -// logic isn't happening here