Browse Source

Revert "fix: Address CA1847 rule"

This reverts commit 2f8273ae64.
pull/9190/head
Giuseppe Lippolis 4 years ago
parent
commit
221bec69d4
  1. 8
      src/Avalonia.Base/AvaloniaProperty.cs
  2. 2
      src/Avalonia.Base/AvaloniaPropertyRegistry.cs
  3. 7
      src/Avalonia.Base/Media/UnicodeRange.cs
  4. 2
      src/Avalonia.Controls/Flyouts/FlyoutBase.cs
  5. 4
      src/Avalonia.Controls/NumericUpDown/NumericUpDown.cs
  6. 14
      src/Avalonia.Remote.Protocol/MetsysBson.cs
  7. 8
      src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/AvaloniaXamlIlLanguageParseIntrinsics.cs
  8. 10
      src/Markup/Avalonia.Markup.Xaml/Converters/TimeSpanTypeConverter.cs

8
src/Avalonia.Base/AvaloniaProperty.cs

@ -12,12 +12,6 @@ namespace Avalonia
/// </summary> /// </summary>
public abstract class AvaloniaProperty : IEquatable<AvaloniaProperty>, IPropertyInfo public abstract class AvaloniaProperty : IEquatable<AvaloniaProperty>, IPropertyInfo
{ {
#if NET6_0_OR_GREATER
internal const char Period = '.';
#else
internal const string Period = ".";
#endif
/// <summary> /// <summary>
/// Represents an unset property value. /// Represents an unset property value.
/// </summary> /// </summary>
@ -47,7 +41,7 @@ namespace Avalonia
{ {
_ = name ?? throw new ArgumentNullException(nameof(name)); _ = name ?? throw new ArgumentNullException(nameof(name));
if (name.Contains(Period)) if (name.Contains("."))
{ {
throw new ArgumentException("'name' may not contain periods."); throw new ArgumentException("'name' may not contain periods.");
} }

2
src/Avalonia.Base/AvaloniaPropertyRegistry.cs

@ -228,7 +228,7 @@ namespace Avalonia
_ = type ?? throw new ArgumentNullException(nameof(type)); _ = type ?? throw new ArgumentNullException(nameof(type));
_ = name ?? throw new ArgumentNullException(nameof(name)); _ = name ?? throw new ArgumentNullException(nameof(name));
if (name.Contains(AvaloniaProperty.Period)) if (name.Contains("."))
{ {
throw new InvalidOperationException("Attached properties not supported."); throw new InvalidOperationException("Attached properties not supported.");
} }

7
src/Avalonia.Base/Media/UnicodeRange.cs

@ -104,11 +104,6 @@ namespace Avalonia.Media
public readonly struct UnicodeRangeSegment public readonly struct UnicodeRangeSegment
{ {
#if NET6_0_OR_GREATER
private const char Question = '?';
#else
private const string Question = "?";
#endif
private static Regex s_regex = new Regex(@"^(?:[uU]\+)?(?:([0-9a-fA-F](?:[0-9a-fA-F?]{1,5})?))$"); private static Regex s_regex = new Regex(@"^(?:[uU]\+)?(?:([0-9a-fA-F](?:[0-9a-fA-F?]{1,5})?))$");
public UnicodeRangeSegment(int start, int end) public UnicodeRangeSegment(int start, int end)
@ -168,7 +163,7 @@ namespace Avalonia.Media
throw new FormatException("Could not parse specified Unicode range segment."); throw new FormatException("Could not parse specified Unicode range segment.");
} }
if (!single.Value.Contains(Question)) if (!single.Value.Contains("?"))
{ {
start = int.Parse(single.Groups[1].Value, System.Globalization.NumberStyles.HexNumber); start = int.Parse(single.Groups[1].Value, System.Globalization.NumberStyles.HexNumber);
end = start; end = start;

2
src/Avalonia.Controls/Flyouts/FlyoutBase.cs

@ -597,7 +597,7 @@ namespace Avalonia.Controls.Primitives
for (int i = presenter.Classes.Count - 1; i >= 0; i--) for (int i = presenter.Classes.Count - 1; i >= 0; i--)
{ {
if (!classes.Contains(presenter.Classes[i]) && if (!classes.Contains(presenter.Classes[i]) &&
!presenter.Classes[i].Contains(':')) !presenter.Classes[i].Contains(":"))
{ {
presenter.Classes.RemoveAt(i); presenter.Classes.RemoveAt(i);
} }

4
src/Avalonia.Controls/NumericUpDown/NumericUpDown.cs

@ -1151,8 +1151,8 @@ namespace Avalonia.Controls
if (PIndex >= 0) if (PIndex >= 0)
{ {
//stringToTest contains a "P" between 2 "'", it's considered as text, not percent //stringToTest contains a "P" between 2 "'", it's considered as text, not percent
var isText = stringToTest.Substring(0, PIndex).Contains('\'') var isText = stringToTest.Substring(0, PIndex).Contains("'")
&& stringToTest.Substring(PIndex, FormatString.Length - PIndex).Contains('\''); && stringToTest.Substring(PIndex, FormatString.Length - PIndex).Contains("'");
return !isText; return !isText;
} }

14
src/Avalonia.Remote.Protocol/MetsysBson.cs

@ -1364,13 +1364,13 @@ namespace Metsys.Bson
var optionsString = ReadName(); var optionsString = ReadName();
var options = RegexOptions.None; var options = RegexOptions.None;
if (optionsString.Contains('e')) options = options | RegexOptions.ECMAScript; if (optionsString.Contains("e")) options = options | RegexOptions.ECMAScript;
if (optionsString.Contains('i')) options = options | RegexOptions.IgnoreCase; if (optionsString.Contains("i")) options = options | RegexOptions.IgnoreCase;
if (optionsString.Contains('l')) options = options | RegexOptions.CultureInvariant; if (optionsString.Contains("l")) options = options | RegexOptions.CultureInvariant;
if (optionsString.Contains('m')) options = options | RegexOptions.Multiline; if (optionsString.Contains("m")) options = options | RegexOptions.Multiline;
if (optionsString.Contains('s')) options = options | RegexOptions.Singleline; if (optionsString.Contains("s")) options = options | RegexOptions.Singleline;
if (optionsString.Contains('w')) options = options | RegexOptions.IgnorePatternWhitespace; if (optionsString.Contains("w")) options = options | RegexOptions.IgnorePatternWhitespace;
if (optionsString.Contains('x')) options = options | RegexOptions.ExplicitCapture; if (optionsString.Contains("x")) options = options | RegexOptions.ExplicitCapture;
return new Regex(pattern, options); return new Regex(pattern, options);
} }

8
src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/AvaloniaXamlIlLanguageParseIntrinsics.cs

@ -14,12 +14,6 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions
{ {
class AvaloniaXamlIlLanguageParseIntrinsics class AvaloniaXamlIlLanguageParseIntrinsics
{ {
#if NET6_0_OR_GREATER
private const string Colon = ":";
#else
private const string Colon = ":";
#endif
public static bool TryConvert(AstTransformationContext context, IXamlAstValueNode node, string text, IXamlType type, AvaloniaXamlIlWellKnownTypes types, out IXamlAstValueNode result) public static bool TryConvert(AstTransformationContext context, IXamlAstValueNode node, string text, IXamlType type, AvaloniaXamlIlWellKnownTypes types, out IXamlAstValueNode result)
{ {
if (type.FullName == "System.TimeSpan") if (type.FullName == "System.TimeSpan")
@ -29,7 +23,7 @@ namespace Avalonia.Markup.Xaml.XamlIl.CompilerExtensions
if (!TimeSpan.TryParse(tsText, CultureInfo.InvariantCulture, out var timeSpan)) if (!TimeSpan.TryParse(tsText, CultureInfo.InvariantCulture, out var timeSpan))
{ {
// // shorthand seconds format (ie. "0.25") // // shorthand seconds format (ie. "0.25")
if (!tsText.Contains(Colon) && double.TryParse(tsText, if (!tsText.Contains(":") && double.TryParse(tsText,
NumberStyles.Float | NumberStyles.AllowThousands, NumberStyles.Float | NumberStyles.AllowThousands,
CultureInfo.InvariantCulture, out var seconds)) CultureInfo.InvariantCulture, out var seconds))
timeSpan = TimeSpan.FromSeconds(seconds); timeSpan = TimeSpan.FromSeconds(seconds);

10
src/Markup/Avalonia.Markup.Xaml/Converters/TimeSpanTypeConverter.cs

@ -7,12 +7,6 @@ namespace Avalonia.Markup.Xaml.Converters
public class TimeSpanTypeConverter : System.ComponentModel.TimeSpanConverter public class TimeSpanTypeConverter : System.ComponentModel.TimeSpanConverter
{ {
#if NET6_0_OR_GREATER
private const char Colon = ':';
#else
private const string Colon = ":";
#endif
public override bool CanConvertFrom(ITypeDescriptorContext context, Type sourceType) public override bool CanConvertFrom(ITypeDescriptorContext context, Type sourceType)
{ {
return base.CanConvertFrom(context, sourceType); return base.CanConvertFrom(context, sourceType);
@ -21,7 +15,7 @@ namespace Avalonia.Markup.Xaml.Converters
public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo culture, object value) public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo culture, object value)
{ {
var valueStr = (string)value; var valueStr = (string)value;
if (!valueStr.Contains(Colon)) if (!valueStr.Contains(":"))
{ {
// shorthand seconds format (ie. "0.25") // shorthand seconds format (ie. "0.25")
var secs = double.Parse(valueStr, CultureInfo.InvariantCulture); var secs = double.Parse(valueStr, CultureInfo.InvariantCulture);
@ -31,4 +25,4 @@ namespace Avalonia.Markup.Xaml.Converters
return base.ConvertFrom(context, culture, value); return base.ConvertFrom(context, culture, value);
} }
} }
} }
Loading…
Cancel
Save