WinGui:

- Some improvements to error messages, about window, minor code style fixes.

git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@3006 b64f7644-9d1e-0410-96f1-a4d463321fa5

6 files changed, 135 insertions, 108 deletions

diff --git a/win/C#/Presets/Import.cs b/win/C#/Presets/Import.cs
index 4d6b1b43a..03f192ea1 100644
--- a/win/C#/Presets/Import.cs
+++ b/win/C#/Presets/Import.cs
@@ -11,25 +11,34 @@ namespace Handbrake.Presets
     {

         public QueryParser importMacPreset(string filename)

         {

-            if (!File.Exists(filename))

-                return null;

+            try

+            {

+                if (!File.Exists(filename))

+                    return null;

 

-            StreamReader sr = File.OpenText(filename);

-            string fromfile = string.Empty;

-            int fileChar;

-            while ((fileChar = sr.Read()) != -1)

-                fromfile += Convert.ToChar(fileChar);

+                StreamReader sr = File.OpenText(filename);

+                string fromfile = string.Empty;

+                int fileChar;

+                while ((fileChar = sr.Read()) != -1)

+                    fromfile += Convert.ToChar(fileChar);

 

-            XmlDocument doc = new XmlDocument();

-            doc.LoadXml(fromfile);

+                XmlDocument doc = new XmlDocument();

+                doc.LoadXml(fromfile);

 

-            XmlNode root = doc;

-            if (!root.HasChildNodes)

+                XmlNode root = doc;

+                if (!root.HasChildNodes)

+                {

+                    MessageBox.Show(

+                        "The Preset file you selected appears to be invlaid or from an older version of HandBrake", "Error",

+                        MessageBoxButtons.OK, MessageBoxIcon.Error);

+                    return null;

+                }

+            }

+            catch (Exception)

             {

                 MessageBox.Show(

-                    "The Preset file you selected appears to be invlaid or from an older version of HandBrake", "Error",

-                    MessageBoxButtons.OK, MessageBoxIcon.Error);

-                return null;

+                "The Preset file you selected appears to be invlaid or from an older version of HandBrake.\n\n Please note, if you are exporting from the MacGui you may need to rebuild your preset so that it uses the current preset plist format.\n The MacGui does not currently update user presets automatically.", "Error",

+                       MessageBoxButtons.OK, MessageBoxIcon.Error);

             }

 

             // We'll query a query parser object and use it's public var structures to store all the data.

@@ -71,7 +80,7 @@ namespace Handbrake.Presets
                             break;

                     }

                 }

-               AudioInfo.Add(track);

+                AudioInfo.Add(track);

             }

             queryParsed.AudioInformation = AudioInfo;

 

diff --git a/win/C#/frmAbout.Designer.cs b/win/C#/frmAbout.Designer.cs
index 2b9c46faa..05cdaeb11 100644
--- a/win/C#/frmAbout.Designer.cs
+++ b/win/C#/frmAbout.Designer.cs
@@ -40,6 +40,8 @@ namespace Handbrake
             this.lbl_HBBuild = new System.Windows.Forms.Label();

             this.PictureBox1 = new System.Windows.Forms.PictureBox();

             this.btn_close = new System.Windows.Forms.Button();

+            this.richTextBox1 = new System.Windows.Forms.RichTextBox();

+            this.label2 = new System.Windows.Forms.Label();

             ((System.ComponentModel.ISupportInitialize)(this.PictureBox1)).BeginInit();

             this.SuspendLayout();

             // 

@@ -93,7 +95,7 @@ namespace Handbrake
             // btn_close

             // 

             this.btn_close.DialogResult = System.Windows.Forms.DialogResult.OK;

-            this.btn_close.Location = new System.Drawing.Point(274, 82);

+            this.btn_close.Location = new System.Drawing.Point(468, 279);

             this.btn_close.Name = "btn_close";

             this.btn_close.Size = new System.Drawing.Size(75, 23);

             this.btn_close.TabIndex = 35;

@@ -101,13 +103,34 @@ namespace Handbrake
             this.btn_close.UseVisualStyleBackColor = true;

             this.btn_close.Click += new System.EventHandler(this.btn_close_Click);

             // 

+            // richTextBox1

+            // 

+            this.richTextBox1.Location = new System.Drawing.Point(128, 111);

+            this.richTextBox1.Name = "richTextBox1";

+            this.richTextBox1.Size = new System.Drawing.Size(415, 162);

+            this.richTextBox1.TabIndex = 36;

+            this.richTextBox1.Text = resources.GetString("richTextBox1.Text");

+            // 

+            // label2

+            // 

+            this.label2.AutoSize = true;

+            this.label2.Location = new System.Drawing.Point(125, 92);

+            this.label2.Margin = new System.Windows.Forms.Padding(3);

+            this.label2.Name = "label2";

+            this.label2.Size = new System.Drawing.Size(46, 13);

+            this.label2.TabIndex = 37;

+            this.label2.Text = "License:";

+            this.label2.TextAlign = System.Drawing.ContentAlignment.MiddleLeft;

+            // 

             // frmAbout

             // 

             this.AcceptButton = this.btn_close;

             this.AutoScaleDimensions = new System.Drawing.SizeF(96F, 96F);

             this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Dpi;

             this.CancelButton = this.btn_close;

-            this.ClientSize = new System.Drawing.Size(363, 117);

+            this.ClientSize = new System.Drawing.Size(555, 314);

+            this.Controls.Add(this.label2);

+            this.Controls.Add(this.richTextBox1);

             this.Controls.Add(this.btn_close);

             this.Controls.Add(this.label1);

             this.Controls.Add(this.PictureBox1);

@@ -136,5 +159,7 @@ namespace Handbrake
         internal System.Windows.Forms.Label lbl_HBBuild;

         internal System.Windows.Forms.PictureBox PictureBox1;

         private System.Windows.Forms.Button btn_close;

+        private System.Windows.Forms.RichTextBox richTextBox1;

+        internal System.Windows.Forms.Label label2;

     }

 }
\ No newline at end of file
diff --git a/win/C#/frmAbout.resx b/win/C#/frmAbout.resx
index 37f92905a..796f0d5e9 100644
--- a/win/C#/frmAbout.resx
+++ b/win/C#/frmAbout.resx
@@ -117,6 +117,13 @@
   <resheader name="writer">

     <value>System.Resources.ResXResourceWriter, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>

   </resheader>

+  <data name="richTextBox1.Text" xml:space="preserve">

+    <value>This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version.
+
+This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License along with this program; if not, write to the Free Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA</value>

+  </data>

   <assembly alias="System.Drawing" name="System.Drawing, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" />

   <data name="$this.Icon" type="System.Drawing.Icon, System.Drawing" mimetype="application/x-microsoft.net.object.bytearray.base64">

     <value>

diff --git a/win/C#/frmActivityWindow.cs b/win/C#/frmActivityWindow.cs
index 61d1b06ca..5f41c799b 100644
--- a/win/C#/frmActivityWindow.cs
+++ b/win/C#/frmActivityWindow.cs
@@ -23,12 +23,12 @@ namespace Handbrake
         private static int _position;

         private static string _lastMode;

         private static string _currentMode;

-        private Thread monitor;

-        private Boolean kilLThread;

+        private Thread _monitor;

+        private Boolean _kilLThread;

 

         public frmActivityWindow(string mode)

         {

-            kilLThread = false;

+            _kilLThread = false;

             _position = 0;

             if (mode == "scan")

                 SetScanMode();

@@ -39,15 +39,15 @@ namespace Handbrake
         }

         private void NewActivityWindow_Load(object sender, EventArgs e)

         {

-            monitor = new Thread(LogMonitor);

+            _monitor = new Thread(LogMonitor);

             

             try

             {

-                monitor.Start();

+                _monitor.Start();

             }

             catch (Exception exc)

             {

-                MessageBox.Show(exc.ToString());

+                MessageBox.Show("Unable to monitor HandBrakes log files. Please report this error. If the problem presists, try rebooting your computer.\n\n Debug Informaton:\n" + exc, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);

             }

         }

 

@@ -55,7 +55,7 @@ namespace Handbrake
         {

             while (true)

             {

-                if (!IsHandleCreated || kilLThread) // break out the thread if the window has been disposed.

+                if (!IsHandleCreated || _kilLThread) // break out the thread if the window has been disposed.

                     break;

 

                 // Perform a reset if require.

@@ -135,7 +135,7 @@ namespace Handbrake
             }

             catch (Exception exc)

             {

-                appendText.AppendFormat("\nERROR: The Log file could not be read. You may need to restart HandBrake! " + exc);

+                appendText.AppendFormat("\n The Log file could not be read. You may need to restart HandBrake! " + exc, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);

                 _position = 0;

                 ClearWindowText();

             }

@@ -161,7 +161,7 @@ namespace Handbrake
             }

             catch (Exception exc)

             {

-                MessageBox.Show("SetWindowText(): Exception: \n" + exc);

+                MessageBox.Show("Unless you are having problems, you can probably ignore this error. It would not hurt to report this error!\n\nSetWindowText(): Exception: \n" + exc, "Error", MessageBoxButtons.OK, MessageBoxIcon.Information);

             }

         }

         private void ClearWindowText()

@@ -181,7 +181,7 @@ namespace Handbrake
             }

             catch (Exception exc)

             {

-                MessageBox.Show("ClearWindowText(): Exception: \n" + exc);

+                MessageBox.Show("Unless you are having problems, you can probably ignore this error. It would not hurt to report this error!\n\nClearWindowText(): Exception: \n" + exc, "Error", MessageBoxButtons.OK, MessageBoxIcon.Information);

             }

         }

         private void PrintLogHeader()

@@ -212,7 +212,7 @@ namespace Handbrake
             }

             catch (Exception exc)

             {

-                MessageBox.Show("PrintLogHeader(): Exception: \n" + exc);

+                MessageBox.Show("Unless you are having problems, you can probably ignore this error. It would not hurt to report this error!\n\nPrintLogHeader(): Exception: \n" + exc, "Error", MessageBoxButtons.OK, MessageBoxIcon.Information);

             }

 

         }

@@ -318,9 +318,9 @@ namespace Handbrake
 

         protected override void OnClosing(CancelEventArgs e)

         {

-            kilLThread = true;

-            monitor.Interrupt();

-            monitor.Join();

+            _kilLThread = true;

+            _monitor.Interrupt();

+            _monitor.Join();

             e.Cancel = true;

             this.Dispose();

             base.OnClosing(e);

diff --git a/win/C#/frmMain.Designer.cs b/win/C#/frmMain.Designer.cs
index 5133bcfe1..57ba75590 100644
--- a/win/C#/frmMain.Designer.cs
+++ b/win/C#/frmMain.Designer.cs
@@ -90,7 +90,6 @@ namespace Handbrake
             this.HelpToolStripMenuItem = new System.Windows.Forms.ToolStripMenuItem();

             this.mnu_user_guide = new System.Windows.Forms.ToolStripMenuItem();

             this.mnu_handbrake_home = new System.Windows.Forms.ToolStripMenuItem();

-            this.mnu_handbrake_forums = new System.Windows.Forms.ToolStripMenuItem();

             this.ToolStripSeparator3 = new System.Windows.Forms.ToolStripSeparator();

             this.mnu_UpdateCheck = new System.Windows.Forms.ToolStripMenuItem();

             this.toolStripSeparator6 = new System.Windows.Forms.ToolStripSeparator();

@@ -100,8 +99,10 @@ namespace Handbrake
             this.Label47 = new System.Windows.Forms.Label();

             this.Label3 = new System.Windows.Forms.Label();

             this.tab_audio = new System.Windows.Forms.TabPage();

+            this.AudioSettings = new Handbrake.Controls.AudioPanel();

             this.AudioMenuRowHeightHack = new System.Windows.Forms.ImageList(this.components);

             this.tab_video = new System.Windows.Forms.TabPage();

+            this.lbl_qualityValue = new System.Windows.Forms.Label();

             this.radio_cq = new System.Windows.Forms.RadioButton();

             this.radio_avgBitrate = new System.Windows.Forms.RadioButton();

             this.radio_targetFilesize = new System.Windows.Forms.RadioButton();

@@ -111,13 +112,17 @@ namespace Handbrake
             this.lbl_SliderValue = new System.Windows.Forms.Label();

             this.Label46 = new System.Windows.Forms.Label();

             this.tab_picture = new System.Windows.Forms.TabPage();

+            this.PictureSettings = new Handbrake.Controls.PictureSettings();

             this.Check_ChapterMarkers = new System.Windows.Forms.CheckBox();

             this.tabs_panel = new System.Windows.Forms.TabControl();

             this.tab_filters = new System.Windows.Forms.TabPage();

+            this.Filters = new Handbrake.Controls.Filters();

             this.tab_subtitles = new System.Windows.Forms.TabPage();

+            this.Subtitles = new Handbrake.Controls.Subtitles();

             this.tab_chapters = new System.Windows.Forms.TabPage();

             this.label31 = new System.Windows.Forms.Label();

             this.tab_advanced = new System.Windows.Forms.TabPage();

+            this.x264Panel = new Handbrake.Controls.x264Panel();

             this.tab_query = new System.Windows.Forms.TabPage();

             this.btn_clear = new System.Windows.Forms.Button();

             this.label34 = new System.Windows.Forms.Label();

@@ -171,12 +176,6 @@ namespace Handbrake
             this.flowLayoutPanel1 = new System.Windows.Forms.FlowLayoutPanel();

             this.openPreset = new System.Windows.Forms.OpenFileDialog();

             this.File_ChapterImport = new System.Windows.Forms.OpenFileDialog();

-            this.lbl_qualityValue = new System.Windows.Forms.Label();

-            this.PictureSettings = new Handbrake.Controls.PictureSettings();

-            this.Filters = new Handbrake.Controls.Filters();

-            this.AudioSettings = new Handbrake.Controls.AudioPanel();

-            this.Subtitles = new Handbrake.Controls.Subtitles();

-            this.x264Panel = new Handbrake.Controls.x264Panel();

             notifyIconMenu = new System.Windows.Forms.ContextMenuStrip(this.components);

             notifyIconMenu.SuspendLayout();

             ((System.ComponentModel.ISupportInitialize)(this.slider_videoQuality)).BeginInit();

@@ -684,7 +683,6 @@ namespace Handbrake
             this.HelpToolStripMenuItem.DropDownItems.AddRange(new System.Windows.Forms.ToolStripItem[] {

             this.mnu_user_guide,

             this.mnu_handbrake_home,

-            this.mnu_handbrake_forums,

             this.ToolStripSeparator3,

             this.mnu_UpdateCheck,

             this.toolStripSeparator6,

@@ -709,13 +707,6 @@ namespace Handbrake
             this.mnu_handbrake_home.Text = "HandBrake Homepage";

             this.mnu_handbrake_home.Click += new System.EventHandler(this.mnu_handbrake_home_Click);

             // 

-            // mnu_handbrake_forums

-            // 

-            this.mnu_handbrake_forums.Name = "mnu_handbrake_forums";

-            this.mnu_handbrake_forums.Size = new System.Drawing.Size(194, 22);

-            this.mnu_handbrake_forums.Text = "HandBrake Forums";

-            this.mnu_handbrake_forums.Click += new System.EventHandler(this.mnu_handbrake_forums_Click);

-            // 

             // ToolStripSeparator3

             // 

             this.ToolStripSeparator3.Name = "ToolStripSeparator3";

@@ -800,6 +791,15 @@ namespace Handbrake
             this.tab_audio.Text = "Audio";

             this.tab_audio.UseVisualStyleBackColor = true;

             // 

+            // AudioSettings

+            // 

+            this.AudioSettings.BackColor = System.Drawing.Color.Transparent;

+            this.AudioSettings.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

+            this.AudioSettings.Location = new System.Drawing.Point(0, 0);

+            this.AudioSettings.Name = "AudioSettings";

+            this.AudioSettings.Size = new System.Drawing.Size(715, 310);

+            this.AudioSettings.TabIndex = 0;

+            // 

             // AudioMenuRowHeightHack

             // 

             this.AudioMenuRowHeightHack.ColorDepth = System.Windows.Forms.ColorDepth.Depth8Bit;

@@ -833,6 +833,16 @@ namespace Handbrake
             this.tab_video.Text = "Video";

             this.tab_video.UseVisualStyleBackColor = true;

             // 

+            // lbl_qualityValue

+            // 

+            this.lbl_qualityValue.AutoSize = true;

+            this.lbl_qualityValue.BackColor = System.Drawing.Color.Transparent;

+            this.lbl_qualityValue.Location = new System.Drawing.Point(534, 100);

+            this.lbl_qualityValue.Name = "lbl_qualityValue";

+            this.lbl_qualityValue.Size = new System.Drawing.Size(24, 13);

+            this.lbl_qualityValue.TabIndex = 19;

+            this.lbl_qualityValue.Text = "0%";

+            // 

             // radio_cq

             // 

             this.radio_cq.AutoSize = true;

@@ -938,6 +948,16 @@ namespace Handbrake
             this.tab_picture.Text = "Picture";

             this.tab_picture.UseVisualStyleBackColor = true;

             // 

+            // PictureSettings

+            // 

+            this.PictureSettings.BackColor = System.Drawing.Color.Transparent;

+            this.PictureSettings.Enabled = false;

+            this.PictureSettings.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

+            this.PictureSettings.Location = new System.Drawing.Point(0, 0);

+            this.PictureSettings.Name = "PictureSettings";

+            this.PictureSettings.Size = new System.Drawing.Size(666, 279);

+            this.PictureSettings.TabIndex = 0;

+            // 

             // Check_ChapterMarkers

             // 

             this.Check_ChapterMarkers.AutoSize = true;

@@ -977,6 +997,15 @@ namespace Handbrake
             this.tab_filters.Text = "Video Filters";

             this.tab_filters.UseVisualStyleBackColor = true;

             // 

+            // Filters

+            // 

+            this.Filters.BackColor = System.Drawing.Color.Transparent;

+            this.Filters.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

+            this.Filters.Location = new System.Drawing.Point(0, 0);

+            this.Filters.Name = "Filters";

+            this.Filters.Size = new System.Drawing.Size(713, 310);

+            this.Filters.TabIndex = 0;

+            // 

             // tab_subtitles

             // 

             this.tab_subtitles.Controls.Add(this.Subtitles);

@@ -988,6 +1017,15 @@ namespace Handbrake
             this.tab_subtitles.Text = "Subtitles";

             this.tab_subtitles.UseVisualStyleBackColor = true;

             // 

+            // Subtitles

+            // 

+            this.Subtitles.BackColor = System.Drawing.Color.Transparent;

+            this.Subtitles.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

+            this.Subtitles.Location = new System.Drawing.Point(0, 0);

+            this.Subtitles.Name = "Subtitles";

+            this.Subtitles.Size = new System.Drawing.Size(722, 310);

+            this.Subtitles.TabIndex = 0;

+            // 

             // tab_chapters

             // 

             this.tab_chapters.BackColor = System.Drawing.Color.Transparent;

@@ -1025,6 +1063,16 @@ namespace Handbrake
             this.tab_advanced.Text = "Advanced";

             this.tab_advanced.UseVisualStyleBackColor = true;

             // 

+            // x264Panel

+            // 

+            this.x264Panel.BackColor = System.Drawing.Color.Transparent;

+            this.x264Panel.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

+            this.x264Panel.Location = new System.Drawing.Point(0, 0);

+            this.x264Panel.Name = "x264Panel";

+            this.x264Panel.Size = new System.Drawing.Size(720, 306);

+            this.x264Panel.TabIndex = 0;

+            this.x264Panel.x264Query = "";

+            // 

             // tab_query

             // 

             this.tab_query.Controls.Add(this.btn_clear);

@@ -1598,63 +1646,6 @@ namespace Handbrake
             // 

             this.File_ChapterImport.Filter = "CSV Files|*.csv";

             // 

-            // lbl_qualityValue

-            // 

-            this.lbl_qualityValue.AutoSize = true;

-            this.lbl_qualityValue.BackColor = System.Drawing.Color.Transparent;

-            this.lbl_qualityValue.Location = new System.Drawing.Point(534, 100);

-            this.lbl_qualityValue.Name = "lbl_qualityValue";

-            this.lbl_qualityValue.Size = new System.Drawing.Size(24, 13);

-            this.lbl_qualityValue.TabIndex = 19;

-            this.lbl_qualityValue.Text = "0%";

-            // 

-            // PictureSettings

-            // 

-            this.PictureSettings.BackColor = System.Drawing.Color.Transparent;

-            this.PictureSettings.Enabled = false;

-            this.PictureSettings.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

-            this.PictureSettings.Location = new System.Drawing.Point(0, 0);

-            this.PictureSettings.Name = "PictureSettings";

-            this.PictureSettings.Size = new System.Drawing.Size(666, 279);

-            this.PictureSettings.TabIndex = 0;

-            // 

-            // Filters

-            // 

-            this.Filters.BackColor = System.Drawing.Color.Transparent;

-            this.Filters.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

-            this.Filters.Location = new System.Drawing.Point(0, 0);

-            this.Filters.Name = "Filters";

-            this.Filters.Size = new System.Drawing.Size(713, 310);

-            this.Filters.TabIndex = 0;

-            // 

-            // AudioSettings

-            // 

-            this.AudioSettings.BackColor = System.Drawing.Color.Transparent;

-            this.AudioSettings.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

-            this.AudioSettings.Location = new System.Drawing.Point(0, 0);

-            this.AudioSettings.Name = "AudioSettings";

-            this.AudioSettings.Size = new System.Drawing.Size(715, 310);

-            this.AudioSettings.TabIndex = 0;

-            // 

-            // Subtitles

-            // 

-            this.Subtitles.BackColor = System.Drawing.Color.Transparent;

-            this.Subtitles.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

-            this.Subtitles.Location = new System.Drawing.Point(0, 0);

-            this.Subtitles.Name = "Subtitles";

-            this.Subtitles.Size = new System.Drawing.Size(722, 310);

-            this.Subtitles.TabIndex = 0;

-            // 

-            // x264Panel

-            // 

-            this.x264Panel.BackColor = System.Drawing.Color.Transparent;

-            this.x264Panel.Font = new System.Drawing.Font("Tahoma", 8.25F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(0)));

-            this.x264Panel.Location = new System.Drawing.Point(0, 0);

-            this.x264Panel.Name = "x264Panel";

-            this.x264Panel.Size = new System.Drawing.Size(720, 306);

-            this.x264Panel.TabIndex = 0;

-            this.x264Panel.x264Query = "";

-            // 

             // frmMain

             // 

             this.AllowDrop = true;

@@ -1793,7 +1784,6 @@ namespace Handbrake
         private System.Windows.Forms.ToolStripMenuItem btn_file_source;

         private System.Windows.Forms.ToolStripMenuItem mnu_delete_preset;

         private System.Windows.Forms.ToolStripMenuItem btn_new_preset;

-        private System.Windows.Forms.ToolStripMenuItem mnu_handbrake_forums;

         private System.Windows.Forms.ToolStripMenuItem mnu_user_guide;

         private System.Windows.Forms.ToolStripDropDownButton btn_source;

         private System.Windows.Forms.ToolStripSeparator toolStripSeparator1;

diff --git a/win/C#/frmMain.cs b/win/C#/frmMain.cs
index ecbd85996..cdd062bb5 100644
--- a/win/C#/frmMain.cs
+++ b/win/C#/frmMain.cs
@@ -386,10 +386,6 @@ namespace Handbrake
         #endregion

 

         #region Help Menu

-        private void mnu_handbrake_forums_Click(object sender, EventArgs e)

-        {

-            Process.Start("http://forum.handbrake.fr/");

-        }

         private void mnu_user_guide_Click(object sender, EventArgs e)

         {

             Process.Start("http://trac.handbrake.fr/wiki/HandBrakeGuide");