Skip to content

Commit 1d40f1c

Browse files
FlorianRapplFlorianRappl
authored
Merge pull request #163 from meziantou/uniformize-cssvalue-comparison
Uniformize CssValue.Equals implementation
2 parents 102cc32 + dbf8aca commit 1d40f1c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

48 files changed

+446
-198
lines changed

‎src/AngleSharp.Css/Values/Composites/CssBackgroundLayerValue.cs‎

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ namespace AngleSharp.Css.Values
33
using AngleSharp.Css.Dom;
44
using AngleSharp.Text;
55
using System;
6+
using System.Collections.Generic;
67

78
sealed class CssBackgroundLayerValue : ICssCompositeValue, IEquatable<CssBackgroundLayerValue>
89
{
@@ -139,11 +140,17 @@ public String CssText
139140
/// <returns>True if both are equal, otherwise false.</returns>
140141
public Boolean Equals(CssBackgroundLayerValue other)
141142
{
142-
return _clip.Equals(other._clip) &&
143-
_repeat.Equals(other._repeat) &&
144-
_attachment.Equals(other._attachment) &&
145-
_origin.Equals(other._origin) &&
146-
_size.Equals(other._size);
143+
if(other is not null)
144+
{
145+
var comparer = EqualityComparer<ICssValue>.Default;
146+
return comparer.Equals(_clip, other._clip) &&
147+
comparer.Equals(_repeat, other._repeat) &&
148+
comparer.Equals(_attachment, other._attachment) &&
149+
comparer.Equals(_origin, other._origin) &&
150+
comparer.Equals(_size, other._size);
151+
}
152+
153+
return false;
147154
}
148155

149156
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssBackgroundLayerValue value && Equals(value);

‎src/AngleSharp.Css/Values/Composites/CssBackgroundSizeValue.cs‎

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ namespace AngleSharp.Css.Values
22
{
33
using AngleSharp.Css.Dom;
44
using System;
5+
using System.Collections.Generic;
56

67
/// <summary>
78
/// Represents a CSS background size definition.
@@ -94,7 +95,16 @@ public String CssText
9495
/// </summary>
9596
/// <param name="other">The other background size.</param>
9697
/// <returns>True if both are equivalent, otherwise false.</returns>
97-
public Boolean Equals(CssBackgroundSizeValue other) => _mode == other._mode && _height.Equals(other._height) && _width.Equals(other._width);
98+
public Boolean Equals(CssBackgroundSizeValue other)
99+
{
100+
if (other is not null)
101+
{
102+
var comparer = EqualityComparer<ICssValue>.Default;
103+
return _mode == other._mode && comparer.Equals(_height, other._height) && comparer.Equals(_width, other._width);
104+
}
105+
106+
return false;
107+
}
98108

99109
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssBackgroundSizeValue value && Equals(value);
100110

‎src/AngleSharp.Css/Values/Composites/CssBackgroundValue.cs‎

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ namespace AngleSharp.Css.Values
22
{
33
using AngleSharp.Css.Dom;
44
using System;
5+
using System.Collections.Generic;
56

67
/// <summary>
78
/// Represents a CSS background definition.
@@ -78,7 +79,13 @@ public String CssText
7879
/// <returns>True if both are equal, otherwise false.</returns>
7980
public Boolean Equals(CssBackgroundValue other)
8081
{
81-
return _color.Equals(other._color) && _layers.Equals(other._layers);
82+
if (other is not null)
83+
{
84+
var comparer = EqualityComparer<ICssValue>.Default;
85+
return comparer.Equals(_color, other._color) && comparer.Equals(_layers, other._layers);
86+
}
87+
88+
return false;
8289
}
8390

8491
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssBackgroundValue value && Equals(value);

‎src/AngleSharp.Css/Values/Composites/CssBorderImageSliceValue.cs‎

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ namespace AngleSharp.Css.Values
33
using AngleSharp.Css.Dom;
44
using AngleSharp.Text;
55
using System;
6+
using System.Collections.Generic;
67

78
/// <summary>
89
/// Represents the CSS border image slice definition.
@@ -124,7 +125,13 @@ public String CssText
124125
/// <returns>True if both are equal, otherwise false.</returns>
125126
public Boolean Equals(CssBorderImageSliceValue other)
126127
{
127-
return _filled == other._filled && _bottom.Equals(other._bottom) && _left.Equals(other._left) && _right.Equals(other._right) && _top.Equals(other._top);
128+
if (other is not null)
129+
{
130+
var comparer = EqualityComparer<ICssValue>.Default;
131+
return _filled == other._filled && comparer.Equals(_bottom, other._bottom) && comparer.Equals(_left, other._left) && comparer.Equals(_right, other._right) && comparer.Equals(_top, other._top);
132+
}
133+
134+
return false;
128135
}
129136

130137
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssBorderImageSliceValue value && Equals(value);

‎src/AngleSharp.Css/Values/Composites/CssBorderImageValue.cs‎

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ namespace AngleSharp.Css.Values
33
using AngleSharp.Css.Dom;
44
using AngleSharp.Text;
55
using System;
6+
using System.Collections.Generic;
67

78
/// <summary>
89
/// Represents a CSS border image definition.
@@ -111,7 +112,7 @@ public String CssText
111112

112113
#endregion
113114

114-
#region
115+
#region Methods
115116

116117
/// <summary>
117118
/// Checks if the current value is equal to the provided one.
@@ -120,11 +121,17 @@ public String CssText
120121
/// <returns>True if both are equal, otherwise false.</returns>
121122
public Boolean Equals(CssBorderImageValue other)
122123
{
123-
return _image.Equals(other._image) &&
124-
_slice.Equals(other._slice) &&
125-
_widths.Equals(other._widths) &&
126-
_outsets.Equals(other._outsets) &&
127-
_repeat.Equals(other._repeat);
124+
if (other is not null)
125+
{
126+
var comparer = EqualityComparer<ICssValue>.Default;
127+
return comparer.Equals(_image, other._image) &&
128+
comparer.Equals(_slice, other._slice) &&
129+
comparer.Equals(_widths, other._widths) &&
130+
comparer.Equals(_outsets, other._outsets) &&
131+
comparer.Equals(_repeat, other._repeat);
132+
}
133+
134+
return false;
128135
}
129136

130137
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssBorderImageValue value && Equals(value);
@@ -137,7 +144,7 @@ ICssValue ICssValue.Compute(ICssComputeContext context)
137144
var offsets = _outsets.Compute(context);
138145
var repeat = _repeat.Compute(context);
139146
return new CssBorderImageValue(image, slice, widths, offsets, repeat);
140-
}
147+
}
141148

142149
#endregion
143150
}

‎src/AngleSharp.Css/Values/Composites/CssBorderRadiusValue.cs‎

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ namespace AngleSharp.Css.Values
22
{
33
using AngleSharp.Css.Dom;
44
using System;
5+
using System.Collections.Generic;
56

67
/// <summary>
78
/// Represents a border radius value.
@@ -74,7 +75,13 @@ public String CssText
7475
/// <returns>True if both are equal, otherwise false.</returns>
7576
public Boolean Equals(CssBorderRadiusValue other)
7677
{
77-
return _horizontal.Equals(other._horizontal) && _vertical.Equals(other._vertical);
78+
if (other is not null)
79+
{
80+
var comparer = EqualityComparer<ICssValue>.Default;
81+
return comparer.Equals(_horizontal, other._horizontal) && comparer.Equals(_vertical, other._vertical);
82+
}
83+
84+
return false;
7885
}
7986

8087
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssBorderRadiusValue value && Equals(value);

‎src/AngleSharp.Css/Values/Composites/CssCursorValue.cs‎

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ namespace AngleSharp.Css.Values
33
using AngleSharp.Css.Dom;
44
using AngleSharp.Text;
55
using System;
6+
using System.Collections.Generic;
67
using System.Linq;
78

89
/// <summary>
@@ -74,22 +75,26 @@ public String CssText
7475
/// <returns>True if both are equal, otherwise false.</returns>
7576
public Boolean Equals(CssCursorValue other)
7677
{
77-
var l = _definitions.Length;
78-
79-
if (_cursor.Equals(other._cursor) && l == other._definitions.Length)
78+
if (other is not null)
8079
{
81-
for (var i = 0; i < l; i++)
82-
{
83-
var a = _definitions[i];
84-
var b = other._definitions[i];
80+
var comparer = EqualityComparer<ICssValue>.Default;
81+
var l = _definitions.Length;
8582

86-
if (!a.Equals(b))
83+
if (comparer.Equals(_cursor, other._cursor) && l == other._definitions.Length)
84+
{
85+
for (var i = 0; i < l; i++)
8786
{
88-
return false;
87+
var a = _definitions[i];
88+
var b = other._definitions[i];
89+
90+
if (!comparer.Equals(a, b))
91+
{
92+
return false;
93+
}
8994
}
90-
}
9195

92-
return true;
96+
return true;
97+
}
9398
}
9499

95100
return false;

‎src/AngleSharp.Css/Values/Composites/CssCustomCursorValue.cs‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ namespace AngleSharp.Css.Values
33
using AngleSharp.Css.Dom;
44
using AngleSharp.Text;
55
using System;
6+
using System.Collections.Generic;
67

78
sealed class CssCustomCursorValue : ICssCompositeValue, IEquatable<CssCustomCursorValue>
89
{
@@ -72,7 +73,14 @@ public String CssText
7273
/// <returns>True if both are equal, otherwise false.</returns>
7374
public Boolean Equals(CssCustomCursorValue other)
7475
{
75-
return _position.Equals(other._position) && _source.Equals(other._source);
76+
if (other is not null)
77+
{
78+
var comparer = EqualityComparer<ICssValue>.Default;
79+
return comparer.Equals(_position, other._position) &&
80+
comparer.Equals(_source, other._source);
81+
}
82+
83+
return false;
7684
}
7785

7886
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssCustomCursorValue value && Equals(value);

‎src/AngleSharp.Css/Values/Composites/CssFontValue.cs‎

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ namespace AngleSharp.Css.Values
33
using AngleSharp.Css.Dom;
44
using AngleSharp.Text;
55
using System;
6+
using System.Collections.Generic;
67

78
/// <summary>
89
/// Represents a CSS font definition.
@@ -145,13 +146,19 @@ public String CssText
145146
/// <returns>True if both are equal, otherwise false.</returns>
146147
public Boolean Equals(CssFontValue other)
147148
{
148-
return _lineHeight.Equals(other._lineHeight) &&
149-
_size.Equals(other._size) &&
150-
_weight.Equals(other._weight) &&
151-
_stretch.Equals(other._stretch) &&
152-
_variant.Equals(other._variant) &&
153-
_style.Equals(other._style) &&
154-
_fontFamilies.Equals(other._fontFamilies);
149+
if (other is not null)
150+
{
151+
var comparer = EqualityComparer<ICssValue>.Default;
152+
return comparer.Equals(_lineHeight, other._lineHeight) &&
153+
comparer.Equals(_size, other._size) &&
154+
comparer.Equals(_weight, other._weight) &&
155+
comparer.Equals(_stretch, other._stretch) &&
156+
comparer.Equals(_variant, other._variant) &&
157+
comparer.Equals(_style, other._style) &&
158+
comparer.Equals(_fontFamilies, other._fontFamilies);
159+
}
160+
161+
return false;
155162
}
156163

157164
Boolean IEquatable<ICssValue>.Equals(ICssValue other) => other is CssFontValue value && Equals(value);

‎src/AngleSharp.Css/Values/Composites/CssGradientStopValue.cs‎

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ namespace AngleSharp.Css.Values
22
{
33
using AngleSharp.Css.Dom;
44
using System;
5+
using System.Collections.Generic;
56

67
/// <summary>
78
/// More information can be found at the W3C:
@@ -69,7 +70,13 @@ public CssGradientStopValue(CssColorValue color, ICssValue location = null)
6970
/// <returns>True if both are equal, otherwise false.</returns>
7071
public Boolean Equals(CssGradientStopValue other)
7172
{
72-
return _color.Equals(other._color) && _location.Equals(other._location);
73+
if (other is not null)
74+
{
75+
var comparer = EqualityComparer<ICssValue>.Default;
76+
return comparer.Equals(_color, other._color) && comparer.Equals(_location, other._location);
77+
}
78+
79+
return false;
7380
}
7481

7582
ICssValue ICssValue.Compute(ICssComputeContext context)

0 commit comments

Comments
 (0)