From 6e9e7804962535504d9511f4956a426da476df42 Mon Sep 17 00:00:00 2001 From: sr55 Date: Fri, 7 Nov 2008 22:28:57 +0000 Subject: [PATCH] WinGui: - Fix for the "frmReadDVD.cs - startProc()System.OverflowException:Arithmetic operation resulted in an overflow" error in Title.cs git-svn-id: svn://localhost/HandBrake/trunk@1903 b64f7644-9d1e-0410-96f1-a4d463321fa5 --- win/C#/Parsing/Title.cs | 30 +++++++----------------------- 1 file changed, 7 insertions(+), 23 deletions(-) diff --git a/win/C#/Parsing/Title.cs b/win/C#/Parsing/Title.cs index c2a21b86..05521c02 100644 --- a/win/C#/Parsing/Title.cs +++ b/win/C#/Parsing/Title.cs @@ -148,7 +148,7 @@ namespace Handbrake.Parsing if (m.Success) thisTitle.m_titleNumber = int.Parse(m.Groups[1].Value.Trim().ToString()); - output.ReadLine(); + String testData = output.ReadLine(); // Get duration for this title @@ -183,29 +183,13 @@ namespace Handbrake.Parsing List titles = new List<Title>(); StringReader sr = new StringReader(output); - while ((char)sr.Peek() == '+') + while (sr.Peek() == '+' || sr.Peek() == ' ') { - titles.Add(Title.Parse(sr)); - /* - * Fix for the line "+ combing detected, may be interlaced or telecined" - * If the next character is not a [ or +, then there are titles left to parse, but we are not where - * we expect to be in the output, so read one line ahead to skip over the unknown line - * - * HACK: FIX THIS PROPERLY - * The try cactch is used to fix a bug introduced with 1819. The level of verbosity needs to be 2 - * in order for the code to run correctly without the hack below. It gets upset about 2 lines of - * log missing with -v 1. To fix this just break out the loop so that sr.Peek() isn't run. - */ - try - { - if ((char)sr.Peek() != '[' && (char)sr.Peek() != '+') // Hack, Fix later - sr.ReadLine(); - } - catch (Exception) - { - break; - } - + // If the the character is a space, then chances are the line + if (sr.Peek() == ' ') // If the character is a space, then chances are it's the combing detected line. + sr.ReadLine(); // Skip over it + else + titles.Add(Title.Parse(sr)); } return titles.ToArray(); -- 2.11.0