You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, Fluent.match(char) will take the character (e.g. $) and wrap it with \Q and \E (e.g. \Q$\E). This is harder to read than \ (e.g. \$). Please make Fluent.match(char) smart in that it will add \ only when necessary.
Furthermore, please make Fluent.match(String) smart in that it will add \ for each character that needs it and if there are 5 or more characters in sequence that need escaping then use \Q and \E. Why 5 or more? To minimize the total length of the regular expression. Adding \Q and \E adds 4 characters. If there are 4 or less characters needing escaping, then adding 4 or less \ will be easier to read.
This will make it easier for humans to read the regular expression.
The text was updated successfully, but these errors were encountered:
Currently,
Fluent.match(char)
will take the character (e.g.$
) and wrap it with\Q
and\E
(e.g.\Q$\E
). This is harder to read than\
(e.g.\$
). Please makeFluent.match(char)
smart in that it will add\
only when necessary.Furthermore, please make
Fluent.match(String)
smart in that it will add\
for each character that needs it and if there are 5 or more characters in sequence that need escaping then use\Q
and\E
. Why 5 or more? To minimize the total length of the regular expression. Adding\Q
and\E
adds 4 characters. If there are 4 or less characters needing escaping, then adding 4 or less\
will be easier to read.This will make it easier for humans to read the regular expression.
The text was updated successfully, but these errors were encountered: