Fix matching new line character at the end of label. Closes smart-pattern/valentina#46.
Because Perl returns a string with a newline at the end when reading a line from a file, Perl’s regex engine matches $ at the position before the line break at the end of the string even when multi-line mode is turned off. Perl also matches $ at the very end of the string, regardless of whether that character is a line break. So ^\d+$ matches 123 whether the subject string is 123 or 123\n. Most modern regex flavors have copied this behavior. That includes .NET, Java, PCRE, Delphi, PHP, and Python. This behavior is independent of any settings such as “multi-line mode”. In all these flavors except Python, \Z also matches before the final line break. If you only want a match at the absolute very end of the string, use \z (lowercase z instead of uppercase Z). \A\d+\z does not match 123\n. \z matches after the line break, which is not matched by the shorthand character class.
This commit is contained in:
parent
2da8f70f86
commit
bdceb76316
|
@ -323,8 +323,11 @@ QString NameRegExp()
|
||||||
//Same regexp in pattern.xsd shema file. Don't forget to synchronize.
|
//Same regexp in pattern.xsd shema file. Don't forget to synchronize.
|
||||||
// \p{Nd} - \p{Decimal_Digit_Number}
|
// \p{Nd} - \p{Decimal_Digit_Number}
|
||||||
// \p{Zs} - \p{Space_Separator}
|
// \p{Zs} - \p{Space_Separator}
|
||||||
regex = QString("^([^\\p{Nd}\\p{Zs}*/&|!<>^\n\\()%1%2%3%4=?:;'\"]){1,1}"
|
// Here we use permanent start of string and end of string anchors \A and \z to match whole pattern as one
|
||||||
"([^\\p{Zs}*/&|!<>^\n\\()%1%2%3%4=?:;\"]){0,}$")
|
// string. In some cases, a user may pass multiline or line that ends with a new line. To cover case with a new
|
||||||
|
// line at the end of string use /z anchor.
|
||||||
|
regex = QString("\\A([^\\p{Nd}\\p{Zs}*/&|!<>^\\n\\()%1%2%3%4=?:;'\"]){1,1}"
|
||||||
|
"([^\\p{Zs}*/&|!<>^\\n\\()%1%2%3%4=?:;\"]){0,}\\z")
|
||||||
.arg(negativeSigns, positiveSigns, decimalPoints, groupSeparators);
|
.arg(negativeSigns, positiveSigns, decimalPoints, groupSeparators);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user