Mirrors/SubtitleEdit
1
0
Fork 0
mirror of https://github.com/SubtitleEdit/subtitleedit.git synced 2024-11-22 03:02:35 +01:00
Code Issues Projects Releases Wiki Activity

Minor refact

Browse Source
This commit is contained in:
niksedk 2021-12-10 22:30:57 +01:00
parent d726027b76
commit 052d72271b
2 changed files with 54 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
LanguageMaster.xml
View File

@ -1506,8 +1506,7 @@ Would you like to start from the top of the document and continue search and rep
<TextingForHearingImpairedRemovedOneLine>Texting for hearing impaired removed: One line</TextingForHearingImpairedRemovedOneLine> <TextingForHearingImpairedRemovedOneLine>Texting for hearing impaired removed: One line</TextingForHearingImpairedRemovedOneLine>
<TextingForHearingImpairedRemovedXLines>Texting for hearing impaired removed: {0} lines</TextingForHearingImpairedRemovedXLines> <TextingForHearingImpairedRemovedXLines>Texting for hearing impaired removed: {0} lines</TextingForHearingImpairedRemovedXLines>
<SubtitleSplitted>Subtitle was split</SubtitleSplitted> <SubtitleSplitted>Subtitle was split</SubtitleSplitted>
<SubtitleAppendPrompt>This will append an existing subtitle to the currently loaded subtitle which should <SubtitleAppendPrompt>This will append an existing subtitle to the currently loaded subtitle which should already be in sync with video file.
already be in sync with video file.
Continue?</SubtitleAppendPrompt> Continue?</SubtitleAppendPrompt>
<SubtitleAppendPromptTitle>Append subtitle</SubtitleAppendPromptTitle> <SubtitleAppendPromptTitle>Append subtitle</SubtitleAppendPromptTitle>

66
src/libse/SubtitleFormats/Pac.cs
View File

@ -1160,7 +1160,7 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
} }
// write text length // write text length
var length = (UInt16)(textBuffer.Length + 4 + (doWritePACHeaderOpt ? PACHeaderOpt.Length : 0)); var length = (ushort)(textBuffer.Length + 4 + (doWritePACHeaderOpt ? PACHeaderOpt.Length : 0));
fs.Write(BitConverter.GetBytes(length), 0, 2); fs.Write(BitConverter.GetBytes(length), 0, 2);
fs.WriteByte(verticalAlignment); // fs.WriteByte(0x0a); // sometimes 0x0b? - this seems to be vertical alignment - 0 to 11 fs.WriteByte(verticalAlignment); // fs.WriteByte(0x0a); // sometimes 0x0b? - this seems to be vertical alignment - 0 to 11
@ -1324,17 +1324,17 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
{ {
subtitle.Paragraphs.Clear(); subtitle.Paragraphs.Clear();
subtitle.Header = null; subtitle.Header = null;
bool usesSecondaryCodePage = UsesSecondaryCodePage(buffer); bool usesSecondaryCodePage = UsesSecondaryCodePage(buffer);
int index = 0; int index = 0;
while (index < buffer.Length) while (index < buffer.Length)
{ {
Paragraph p = GetPacParagraph(ref index, buffer, usesSecondaryCodePage); var p = GetPacParagraph(ref index, buffer, usesSecondaryCodePage, subtitle.Paragraphs.Count);
if (p != null) if (p != null)
{ {
subtitle.Paragraphs.Add(p); subtitle.Paragraphs.Add(p);
} }
} }
subtitle.Renumber(); subtitle.Renumber();
} }
@ -1364,20 +1364,29 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
private static bool CompareBytes(byte[] buff, int pos, byte[] seq) private static bool CompareBytes(byte[] buff, int pos, byte[] seq)
{ {
if (buff.Length < pos + seq.Length) if (buff.Length < pos + seq.Length)
{
return false; return false;
}
for (int i = 0; i < seq.Length; i++) for (int i = 0; i < seq.Length; i++)
{
if (buff[pos + i] != seq[i]) if (buff[pos + i] != seq[i])
{
return false; return false;
}
}
return true; return true;
} }
private Paragraph GetPacParagraph(ref int index, byte[] buffer, bool usesSecondaryCodePage) private Paragraph GetPacParagraph(ref int index, byte[] buffer, bool usesSecondaryCodePage, int paragraphIndex)
{ {
bool isStory = index < 15; var isStory = index < 15;
if (isStory) if (isStory)
{ {
index = 15; index = 15;
} }
while (true) while (true)
{ {
index++; index++;
@ -1386,25 +1395,28 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
return null; return null;
} }
if (buffer[index] == 0xFE && (buffer[index - 15] == 0x60 || buffer[index - 15] == 0x61)) if (buffer[index] == 0xFE)
{
var minus15 = buffer[index - 15];
if (minus15 == 0x60 || minus15 == 0x61 || minus15 == 0x62)
{ {
break; break;
} }
if (buffer[index] == 0xFE && (buffer[index - 12] == 0x60 || buffer[index - 12] == 0x61)) var minus12 = buffer[index - 12];
if (minus12 == 0x60 || minus12 == 0x61 || minus12 == 0x62)
{ {
break; break;
} }
} }
}
int feIndex = index; int feIndex = index;
const int endDelimiter = 0x00; const int endDelimiter = 0x00;
byte alignment = buffer[feIndex + 1]; byte alignment = buffer[feIndex + 1];
bool isSecondaryCodePage = (alignment & 0x08) != 0; bool isSecondaryCodePage = (alignment & 0x08) != 0;
alignment &= 0x07; alignment &= 0x07;
var p = new Paragraph(); var p = new Paragraph();
int timeStartIndex = feIndex - 15; int timeStartIndex = feIndex - 15;
if (buffer[timeStartIndex] == 0x60) if (buffer[timeStartIndex] == 0x60)
{ {
@ -1417,27 +1429,27 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
p.StartTime = GetTimeCode(timeStartIndex + 1, buffer); p.StartTime = GetTimeCode(timeStartIndex + 1, buffer);
p.EndTime = GetTimeCode(timeStartIndex + 5, buffer); p.EndTime = GetTimeCode(timeStartIndex + 5, buffer);
} }
else if (buffer[timeStartIndex] == 0x61) else if (buffer[timeStartIndex] == 0x61 || buffer[timeStartIndex] == 0x62)
{ {
p.StartTime = GetTimeCode(timeStartIndex + 1, buffer); p.StartTime = GetTimeCode(timeStartIndex + 1, buffer);
p.EndTime = GetTimeCode(timeStartIndex + 5, buffer); p.EndTime = GetTimeCode(timeStartIndex + 5, buffer);
int length = buffer[timeStartIndex + 9] + buffer[timeStartIndex + 10] * 256; int length = buffer[timeStartIndex + 9] + buffer[timeStartIndex + 10] * 256;
if (length < 1 || length > 200 || if (length < 1 || length > 200 || (paragraphIndex != 1 &&
p.StartTime.TotalSeconds - _lastStartTotalSeconds < 1 || p.StartTime.TotalSeconds - _lastStartTotalSeconds > 1500 || (p.StartTime.TotalSeconds - _lastStartTotalSeconds < 1 || p.StartTime.TotalSeconds - _lastStartTotalSeconds > 1500 ||
p.EndTime.TotalSeconds - _lastEndTotalSeconds < 1 || p.EndTime.TotalSeconds - _lastEndTotalSeconds > 1500) p.EndTime.TotalSeconds - _lastEndTotalSeconds < 1 || p.EndTime.TotalSeconds - _lastEndTotalSeconds > 1500)))
{ {
return null; return null;
} }
} }
else if (buffer[timeStartIndex + 3] == 0x61) else if (buffer[timeStartIndex + 3] == 0x61 || buffer[timeStartIndex + 3] == 0x62)
{ {
timeStartIndex += 3; timeStartIndex += 3;
p.StartTime = GetTimeCode(timeStartIndex + 1, buffer); p.StartTime = GetTimeCode(timeStartIndex + 1, buffer);
p.EndTime = GetTimeCode(timeStartIndex + 5, buffer); p.EndTime = GetTimeCode(timeStartIndex + 5, buffer);
int length = buffer[timeStartIndex + 9] + buffer[timeStartIndex + 10] * 256; int length = buffer[timeStartIndex + 9] + buffer[timeStartIndex + 10] * 256;
if (length < 1 || length > 200 || if (length < 1 || length > 200 || (paragraphIndex != 1 &&
p.StartTime.TotalSeconds - _lastStartTotalSeconds < 1 || p.StartTime.TotalSeconds - _lastStartTotalSeconds > 1500 || (p.StartTime.TotalSeconds - _lastStartTotalSeconds < 1 || p.StartTime.TotalSeconds - _lastStartTotalSeconds > 1500 ||
p.EndTime.TotalSeconds - _lastEndTotalSeconds < 1 || p.EndTime.TotalSeconds - _lastEndTotalSeconds > 1500) p.EndTime.TotalSeconds - _lastEndTotalSeconds < 1 || p.EndTime.TotalSeconds - _lastEndTotalSeconds > 1500)))
{ {
return null; return null;
} }
@ -1446,6 +1458,7 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
{ {
return null; return null;
} }
int textLength = buffer[timeStartIndex + 9] + buffer[timeStartIndex + 10] * 256; int textLength = buffer[timeStartIndex + 9] + buffer[timeStartIndex + 10] * 256;
if (textLength > 500) if (textLength > 500)
{ {
@ -1548,19 +1561,31 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
else if (isUnicode) else if (isUnicode)
{ {
if (buffer[index] == MarkerEndOfUnicode) if (buffer[index] == MarkerEndOfUnicode)
{
isUnicode = false; isUnicode = false;
}
else if (buffer[index] == MarkerReplaceEndOfUnicode) else if (buffer[index] == MarkerReplaceEndOfUnicode)
{
sb.Append((char)MarkerEndOfUnicode); sb.Append((char)MarkerEndOfUnicode);
}
else else
{ {
int len = 1; int len = 1;
byte b = buffer[index]; byte b = buffer[index];
if (b >= 0xE0) if (b >= 0xE0)
{
len = 3; len = 3;
}
else if (b >= 0xC0) else if (b >= 0xC0)
{
len = 2; len = 2;
}
if (buffer.Length > index + len - 1) if (buffer.Length > index + len - 1)
{
sb.Append(Encoding.UTF8.GetString(buffer, index, len)); sb.Append(Encoding.UTF8.GetString(buffer, index, len));
}
index += len - 1; index += len - 1;
} }
} }
@ -1600,6 +1625,7 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
index++; index++;
} }
if (index + 20 >= buffer.Length) if (index + 20 >= buffer.Length)
{ {
return null; return null;
@ -1658,7 +1684,8 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
p.Text = "{\\an3}" + p.Text; p.Text = "{\\an3}" + p.Text;
} }
} }
p.Text = p.Text.RemoveControlCharactersButWhiteSpace();
p.Text = p.Text.RemoveControlCharactersButWhiteSpace().TrimEnd();
return p; return p;
} }
@ -1809,6 +1836,7 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
{ {
// ignored // ignored
} }
return CodePageLatin; return CodePageLatin;
} }
@ -1860,6 +1888,7 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
index++; index++;
} }
} }
previewBuffer = new byte[textIndex]; previewBuffer = new byte[textIndex];
for (int i = 0; i < textIndex; i++) for (int i = 0; i < textIndex; i++)
{ {
@ -2331,6 +2360,7 @@ namespace Nikse.SubtitleEdit.Core.SubtitleFormats
{ {
throw new InvalidOperationException($"Unknown byte ({b}) in subtitle file @ binary offset {index}."); throw new InvalidOperationException($"Unknown byte ({b}) in subtitle file @ binary offset {index}.");
} }
return string.Empty; return string.Empty;
} }