Merge pull request #7897 from PWillumsen/main

Fix: comment and and context was switched for some checks
This commit is contained in:
Nikolaj Olsson 2024-02-03 18:28:05 +01:00 committed by GitHub
commit 1ae1b08d20
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 3 additions and 3 deletions

View File

@ -27,7 +27,7 @@ namespace Nikse.SubtitleEdit.Core.NetflixQualityCheck
if (CalculateJapaneseLength(text) > 11)
{
var comment = "Single vertical line length > 11";
controller.AddRecord(p, p.StartTime.ToHHMMSSFF(), comment, line.Length.ToString(CultureInfo.InvariantCulture));
controller.AddRecord(p, p.StartTime.ToHHMMSSFF(), line.Length.ToString(CultureInfo.InvariantCulture), comment);
}
}
else // Horizontal subtitles - Maximum 13 full-width characters per line
@ -35,7 +35,7 @@ namespace Nikse.SubtitleEdit.Core.NetflixQualityCheck
if (CalculateJapaneseLength(text) > 13)
{
var comment = "Single horizontal line length > 13";
controller.AddRecord(p, p.StartTime.ToHHMMSSFF(), comment, line.Length.ToString(CultureInfo.InvariantCulture));
controller.AddRecord(p, p.StartTime.ToHHMMSSFF(), line.Length.ToString(CultureInfo.InvariantCulture), comment);
}
}
}

View File

@ -19,7 +19,7 @@ namespace Nikse.SubtitleEdit.Core.NetflixQualityCheck
if (p.DurationTotalMilliseconds < 500)
{
string comment = "Minimum duration: 0.5 second";
controller.AddRecord(p, p.StartTime.ToHHMMSSFF(), comment, p.DurationTotalSeconds.ToString(CultureInfo.InvariantCulture));
controller.AddRecord(p, p.StartTime.ToHHMMSSFF(), p.DurationTotalSeconds.ToString(CultureInfo.InvariantCulture), comment);
}
continue;
}