system/coreutils
system/coreutils
1
0
Fork 0
mirror of https://github.com/uutils/coreutils synced 2024-10-15 04:14:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6036 from cj-zoltan-kiss/zoltankiss/globfix

Browse source 
parser: if closing square bracket not found, stop looking for it again
This commit is contained in:
Sylvestre Ledru 2024-03-11 19:51:18 +01:00 committed by GitHub
parent 0da5a7885b be24742282
commit 93b1abff39
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/uucore/src/lib/parser/parse_glob.rs
View file

@ -18,7 +18,11 @@ fn fix_negation(glob: &str) -> String {
while i + 3 < chars.len() {
if chars[i] == '[' && chars[i + 1] == '^' {
match chars[i + 3..].iter().position(|x| *x == ']') {
None => (),
None => {
// if closing square bracket not found, stop looking for it
// again
break;
}
Some(j) => {
chars[i + 1] = '!';
i += j + 4;
@ -90,6 +94,11 @@ mod tests {
assert_eq!(fix_negation("[[]] [^a]"), "[[]] [!a]");
assert_eq!(fix_negation("[[] [^a]"), "[[] [!a]");
assert_eq!(fix_negation("[]] [^a]"), "[]] [!a]");
// test that we don't look for closing square brackets unnecessarily
// Verifies issue #5584
let chars = std::iter::repeat("^[").take(174571).collect::<String>();
assert_eq!(fix_negation(chars.as_str()), chars);
}
#[test]