maxmustermann/PowerShell
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Improve performance of parsing RegexOption for '-split' by using if branches (#6605)

Browse source
This commit is contained in:
Ilya 2018-04-11 00:03:41 +04:00 committed by Dongbo Wang
parent 3cd2308af9
commit 68b59e3d9b
1 changed files with 22 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

36
src/System.Management.Automation/engine/lang/parserutils.cs
View file

@ -529,22 +529,30 @@ namespace System.Management.Automation
private static RegexOptions parseRegexOptions(SplitOptions options)
{
int[][] map = {
new int[] { (int)SplitOptions.CultureInvariant, (int)RegexOptions.CultureInvariant },
new int[] { (int)SplitOptions.IgnorePatternWhitespace, (int)RegexOptions.IgnorePatternWhitespace },
new int[] { (int)SplitOptions.Multiline, (int)RegexOptions.Multiline },
new int[] { (int)SplitOptions.Singleline, (int)RegexOptions.Singleline },
new int[] { (int)SplitOptions.IgnoreCase, (int)RegexOptions.IgnoreCase },
new int[] { (int)SplitOptions.ExplicitCapture, (int)RegexOptions.ExplicitCapture },
};
RegexOptions result = RegexOptions.None;
foreach (int[] entry in map)
if ((options & SplitOptions.CultureInvariant) != 0)
{
if (((int)options & entry[0]) != 0)
{
result |= (RegexOptions)entry[1];
}
result |= RegexOptions.CultureInvariant;
}
if ((options & SplitOptions.IgnorePatternWhitespace) != 0)
{
result |= RegexOptions.IgnorePatternWhitespace;
}
if ((options & SplitOptions.Multiline) != 0)
{
result |= RegexOptions.Multiline;
}
if ((options & SplitOptions.Singleline) != 0)
{
result |= RegexOptions.Singleline;
}
if ((options & SplitOptions.IgnoreCase) != 0)
{
result |= RegexOptions.IgnoreCase;
}
if ((options & SplitOptions.ExplicitCapture) != 0)
{
result |= RegexOptions.ExplicitCapture;
}
return result;