summaryrefslogtreecommitdiffstats
path: root/win
diff options
context:
space:
mode:
authorsr55 <[email protected]>2013-03-20 20:35:48 +0000
committersr55 <[email protected]>2013-03-20 20:35:48 +0000
commit285a71750a2c3fcb5c34d3fbf28e5105c03d7b9a (patch)
treec5c6bac2cc4eaa3a8edc52b7b4ec8e47cf229480 /win
parent97a88523f2b6352847d705cbf527fd5ee0f5247b (diff)
WinGui: Appcast related fixes and tidyup.
git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@5351 b64f7644-9d1e-0410-96f1-a4d463321fa5
Diffstat (limited to 'win')
-rw-r--r--win/CS/HandBrake.ApplicationServices/Utilities/VersionHelper.cs11
-rw-r--r--win/CS/HandBrakeWPF/Constants.cs27
-rw-r--r--win/CS/HandBrakeWPF/HandBrakeWPF.csproj1
-rw-r--r--win/CS/HandBrakeWPF/Helpers/CliCheckHelper.cs9
-rw-r--r--win/CS/HandBrakeWPF/Services/UpdateService.cs7
-rw-r--r--win/CS/HandBrakeWPF/UserSettingConstants.cs15
-rw-r--r--win/CS/HandBrakeWPF/defaultsettings.xml25
7 files changed, 42 insertions, 53 deletions
diff --git a/win/CS/HandBrake.ApplicationServices/Utilities/VersionHelper.cs b/win/CS/HandBrake.ApplicationServices/Utilities/VersionHelper.cs
index a8860f5ee..b1a90d3e0 100644
--- a/win/CS/HandBrake.ApplicationServices/Utilities/VersionHelper.cs
+++ b/win/CS/HandBrake.ApplicationServices/Utilities/VersionHelper.cs
@@ -53,5 +53,16 @@ namespace HandBrake.ApplicationServices.Utilities
{
return System.Environment.Is64BitProcess ? "64bit Version" : "32bit Version";
}
+
+ /// <summary>
+ /// Is a 64 bit app.
+ /// </summary>
+ /// <returns>
+ /// The <see cref="string"/>.
+ /// </returns>
+ public static bool Is64Bit()
+ {
+ return System.Environment.Is64BitProcess;
+ }
}
}
diff --git a/win/CS/HandBrakeWPF/Constants.cs b/win/CS/HandBrakeWPF/Constants.cs
new file mode 100644
index 000000000..862f08011
--- /dev/null
+++ b/win/CS/HandBrakeWPF/Constants.cs
@@ -0,0 +1,27 @@
+// --------------------------------------------------------------------------------------------------------------------
+// <copyright file="Constants.cs" company="HandBrake Project (http://handbrake.fr)">
+// This file is part of the HandBrake source code - It may be used under the terms of the GNU General Public License.
+// </copyright>
+// <summary>
+// Application Constants
+// </summary>
+// --------------------------------------------------------------------------------------------------------------------
+
+namespace HandBrakeWPF
+{
+ /// <summary>
+ /// Application Constants
+ /// </summary>
+ public class Constants
+ {
+ /// <summary>
+ /// The appcast 64.
+ /// </summary>
+ public const string Appcast64 = "http://handbrake.fr/appcast.x86_64.xml";
+
+ /// <summary>
+ /// The appcast 32.
+ /// </summary>
+ public const string Appcast32 = "http://handbrake.fr/appcast.i386.xml";
+ }
+}
diff --git a/win/CS/HandBrakeWPF/HandBrakeWPF.csproj b/win/CS/HandBrakeWPF/HandBrakeWPF.csproj
index 09270768b..d7c81d861 100644
--- a/win/CS/HandBrakeWPF/HandBrakeWPF.csproj
+++ b/win/CS/HandBrakeWPF/HandBrakeWPF.csproj
@@ -130,6 +130,7 @@
<Compile Include="Commands\ProcessShortcutCommand.cs" />
<Compile Include="Commands\SourceMenuCommand.cs" />
<Compile Include="Commands\AdvancedEncoderOptionsCommand.cs" />
+ <Compile Include="Constants.cs" />
<Compile Include="Controls\TimeSpanBox.xaml.cs">
<DependentUpon>TimeSpanBox.xaml</DependentUpon>
</Compile>
diff --git a/win/CS/HandBrakeWPF/Helpers/CliCheckHelper.cs b/win/CS/HandBrakeWPF/Helpers/CliCheckHelper.cs
index cfafe3cec..2e69eacec 100644
--- a/win/CS/HandBrakeWPF/Helpers/CliCheckHelper.cs
+++ b/win/CS/HandBrakeWPF/Helpers/CliCheckHelper.cs
@@ -77,10 +77,8 @@ namespace HandBrakeWPF.Helpers
while ((line = stdOutput.ReadLine()) != null)
{
Match m = Regex.Match(line, @"HandBrake ([svnM0-9.]*) \(([0-9]*)\)");
- Match platform = Regex.Match(line, @"- ([A-Za-z0-9\s ]*) -");
if (m.Success)
{
- string version = m.Groups[1].Success ? m.Groups[1].Value : string.Empty;
string build = m.Groups[2].Success ? m.Groups[2].Value : string.Empty;
int buildValue;
@@ -89,12 +87,6 @@ namespace HandBrakeWPF.Helpers
userSettingService.SetUserSetting(ASUserSettingConstants.HandBrakeBuild, buildValue);
success = true;
}
-
- if (platform.Success)
- {
- userSettingService.SetUserSetting(
- UserSettingConstants.HandBrakePlatform, platform.Value.Replace("-", string.Empty).Trim());
- }
}
while (!cliProcess.HasExited)
@@ -117,7 +109,6 @@ namespace HandBrakeWPF.Helpers
catch (Exception e)
{
userSettingService.SetUserSetting(ASUserSettingConstants.HandBrakeBuild, 0);
- userSettingService.SetUserSetting(UserSettingConstants.HandBrakePlatform, string.Empty);
userSettingService.SetUserSetting(UserSettingConstants.HandBrakeExeHash, string.Empty);
errorService.ShowError(
diff --git a/win/CS/HandBrakeWPF/Services/UpdateService.cs b/win/CS/HandBrakeWPF/Services/UpdateService.cs
index 112493ac7..b95a1d862 100644
--- a/win/CS/HandBrakeWPF/Services/UpdateService.cs
+++ b/win/CS/HandBrakeWPF/Services/UpdateService.cs
@@ -87,10 +87,9 @@ namespace HandBrakeWPF.Services
try
{
string url =
- this.userSettingService.GetUserSetting<string>(UserSettingConstants.HandBrakePlatform)
- .Contains("x86_64")
- ? this.userSettingService.GetUserSetting<string>(UserSettingConstants.Appcast_x64)
- : this.userSettingService.GetUserSetting<string>(UserSettingConstants.Appcast_i686);
+ VersionHelper.Is64Bit()
+ ? Constants.Appcast64
+ : Constants.Appcast32;
var currentBuild =
this.userSettingService.GetUserSetting<int>(ASUserSettingConstants.HandBrakeBuild);
diff --git a/win/CS/HandBrakeWPF/UserSettingConstants.cs b/win/CS/HandBrakeWPF/UserSettingConstants.cs
index 030cdedac..a90094cb5 100644
--- a/win/CS/HandBrakeWPF/UserSettingConstants.cs
+++ b/win/CS/HandBrakeWPF/UserSettingConstants.cs
@@ -22,16 +22,6 @@ namespace HandBrakeWPF
public const string AddOnlyOneAudioPerLanguage = "addOnlyOneAudioPerLanguage";
/// <summary>
- /// Appcast i686
- /// </summary>
- public const string Appcast_i686 = "appcast_i686";
-
- /// <summary>
- /// Appcast 64
- /// </summary>
- public const string Appcast_x64 = "appcast_x64";
-
- /// <summary>
/// Auto name format
/// </summary>
public const string AutoNameFormat = "autoNameFormat";
@@ -197,11 +187,6 @@ namespace HandBrakeWPF
public const string X264Step = "X264Step";
/// <summary>
- /// HandBrakes build
- /// </summary>
- public const string HandBrakePlatform = "HandBrakePlatform";
-
- /// <summary>
/// The show advanced tab.
/// </summary>
public const string ShowAdvancedTab = "ShowAdvancedTab";
diff --git a/win/CS/HandBrakeWPF/defaultsettings.xml b/win/CS/HandBrakeWPF/defaultsettings.xml
index 75810a442..690930721 100644
--- a/win/CS/HandBrakeWPF/defaultsettings.xml
+++ b/win/CS/HandBrakeWPF/defaultsettings.xml
@@ -154,22 +154,6 @@
</item>
<item>
<key>
- <string>appcast_i686</string>
- </key>
- <value>
- <anyType xmlns:q1="http://www.w3.org/2001/XMLSchema" d4p1:type="q1:string" xmlns:d4p1="http://www.w3.org/2001/XMLSchema-instance">http://handbrake.fr/appcast.i386.xml</anyType>
- </value>
- </item>
- <item>
- <key>
- <string>appcast_x64</string>
- </key>
- <value>
- <anyType xmlns:q1="http://www.w3.org/2001/XMLSchema" d4p1:type="q1:string" xmlns:d4p1="http://www.w3.org/2001/XMLSchema-instance">http://handbrake.fr/appcast.86_64.xml</anyType>
- </value>
- </item>
- <item>
- <key>
<string>autoNameFormat</string>
</key>
<value>
@@ -402,21 +386,12 @@
</item>
<item>
<key>
- <string>HandBrakePlatform</string>
- </key>
- <value>
- <anyType xmlns:q1="http://www.w3.org/2001/XMLSchema" d4p1:type="q1:string" xmlns:d4p1="http://www.w3.org/2001/XMLSchema-instance">MinGW i686</anyType>
- </value>
- </item>
- <item>
- <key>
<string>ClearCompletedFromQueue</string>
</key>
<value>
<anyType xmlns:q1="http://www.w3.org/2001/XMLSchema" d4p1:type="q1:boolean" xmlns:d4p1="http://www.w3.org/2001/XMLSchema-instance">false</anyType>
</value>
</item>
-
<item>
<key>
<string>ServerPort</string>