Skip to content

Commit 5a2b500

Browse files
committed
RECS0119: 'string.Compare' is culture-aware
[ci skip]
1 parent 178f9e5 commit 5a2b500

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

ICSharpCode.SharpZipLib.Tests/Zip/ZipTests.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1237,10 +1237,10 @@ public void NameTransforms()
12371237
[Category("Zip")]
12381238
public void FilenameCleaning()
12391239
{
1240-
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName("hello"), "hello"));
1241-
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName(@"z:\eccles"), "eccles"));
1242-
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName(@"\\server\share\eccles"), "eccles"));
1243-
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName(@"\\server\share\dir\eccles"), "dir/eccles"));
1240+
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName("hello"), "hello", StringComparison.Ordinal));
1241+
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName(@"z:\eccles"), "eccles", StringComparison.Ordinal));
1242+
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName(@"\\server\share\eccles"), "eccles", StringComparison.Ordinal));
1243+
Assert.AreEqual(0, string.Compare(ZipEntry.CleanName(@"\\server\share\dir\eccles"), "dir/eccles", StringComparison.Ordinal));
12441244
}
12451245

12461246
[Test]
@@ -3474,18 +3474,18 @@ public void FindEntry()
34743474

34753475
int testIndex = zipFile.FindEntry("Farriera", false);
34763476
Assert.AreEqual(0, testIndex, "Case sensitive find failure");
3477-
Assert.IsTrue(string.Compare(zipFile[testIndex].Name, "Farriera", false) == 0);
3477+
Assert.IsTrue(string.Compare(zipFile[testIndex].Name, "Farriera", StringComparison.Ordinal) == 0);
34783478

34793479
testIndex = zipFile.FindEntry("Farriera", true);
34803480
Assert.AreEqual(0, testIndex, "Case insensitive find failure");
3481-
Assert.IsTrue(string.Compare(zipFile[testIndex].Name, "Farriera", true) == 0);
3481+
Assert.IsTrue(string.Compare(zipFile[testIndex].Name, "Farriera", StringComparison.OrdinalIgnoreCase) == 0);
34823482

34833483
testIndex = zipFile.FindEntry("urban mYTH", false);
34843484
Assert.AreEqual(-1, testIndex, "Case sensitive find failure");
34853485

34863486
testIndex = zipFile.FindEntry("urban mYTH", true);
34873487
Assert.AreEqual(2, testIndex, "Case insensitive find failure");
3488-
Assert.IsTrue(string.Compare(zipFile[testIndex].Name, "urban mYTH", true) == 0);
3488+
Assert.IsTrue(string.Compare(zipFile[testIndex].Name, "urban mYTH", StringComparison.OrdinalIgnoreCase) == 0);
34893489

34903490
testIndex = zipFile.FindEntry("Champane.", false);
34913491
Assert.AreEqual(-1, testIndex, "Case sensitive find failure");

0 commit comments

Comments
 (0)