SEGIMAP icon indicating copy to clipboard operation
SEGIMAP copied to clipboard

Remove uses of regex library

Open indiv0 opened this issue 8 years ago • 2 comments

This line will contain our last regex usage once #24 lands. We should compare the performance to a nom-based parser (if it fits in this use-case), and remove the regex lib if necessary.

This would let us remove the regex library as a dependency entirely.

indiv0 avatar May 20 '17 03:05 indiv0

You missed a spot in src/util.rs.

The one you missed is fairly easy to replace with String::starts_with

The one you highlight in the session submodule of the user module is rather difficult to replace with string operations. It contains optional path separators between the components that it is looking for so it would need to be replaced with 4 strings (without either separators, with the first, with the second, with both separator).

Closing as WONTFIX; re-open if you have any objections to or see any flaws in my reasoning.

uiri avatar May 20 '17 04:05 uiri

With string-based operations it might be difficult, but I think it's still doable with nom. I'd still like to at least give it a try.

indiv0 avatar May 20 '17 12:05 indiv0