WinGui:

- Added some new events to the Queue for future refactoring.
- Refactor / Cleanup / Remove more stylecop warnings.

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

3 files changed, 24 insertions, 26 deletions

diff --git a/win/C#/Controls/Filters.cs b/win/C#/Controls/Filters.cs
index c8427bb98..9f75f69d3 100644
--- a/win/C#/Controls/Filters.cs
+++ b/win/C#/Controls/Filters.cs
@@ -1,5 +1,4 @@
 /*  Filters.cs $

- 	

     This file is part of the HandBrake source code.

     Homepage: <http://handbrake.fr>.

     It may be used under the terms of the GNU General Public License. */

@@ -30,7 +29,7 @@ namespace Handbrake.Controls
         }

 

         /// <summary>

-        /// Returns the CLI query for the query generator.

+        /// Gets the CLI query for the query generator.

         /// </summary>

         public string GetCliQuery

         {

@@ -54,7 +53,6 @@ namespace Handbrake.Controls
                         break;

                 }

 

-

                 switch (drop_decomb.Text) // Decomb

                 {

                     case "Off":

diff --git a/win/C#/Controls/Subtitles.cs b/win/C#/Controls/Subtitles.cs
index 3326ee74e..fc9315757 100644
--- a/win/C#/Controls/Subtitles.cs
+++ b/win/C#/Controls/Subtitles.cs
@@ -84,7 +84,6 @@ namespace Handbrake.Controls
                                    ? srtLangVal + " (" + srtFile + ")"

                                    : drp_subtitleTracks.SelectedItem.ToString();

 

-

             SubtitleInfo track = new SubtitleInfo

                                      {

                                          Track = trackName, 

@@ -433,7 +432,7 @@ namespace Handbrake.Controls
         /// Automatically setup the subtitle tracks.

         /// This handles the automatic setup of subitles which the user can control from the program options

         /// </summary>

-        /// <param name="subs"></param>

+        /// <param name="subs">List of Subtitles</param>

         public void SetSubtitleTrackAuto(object[] subs)

         {

             drp_subtitleTracks.Items.Clear();

@@ -464,7 +463,7 @@ namespace Handbrake.Controls
         /// <summary>

         /// Set the file container which is currently in use.

         /// </summary>

-        /// <param name="value"></param>

+        /// <param name="value">The File Container</param>

         public void SetContainer(int value)

         {

             fileContainer = value;

@@ -484,16 +483,16 @@ namespace Handbrake.Controls
         /// <summary>

         /// Get the list of subtitles.

         /// </summary>

-        /// <returns></returns>

+        /// <returns>A List of SubtitleInfo Object</returns>

         public List<SubtitleInfo> GetSubtitleInfoList()

         {

             return subList;

         }

 

         /// <summary>

-        /// Get the CLI Query for this panel

+        /// Gets the CLI Query for this panel

         /// </summary>

-        /// <returns></returns>

+        /// <returns>A CliQuery string</returns>

         public string GetCliQuery

         {

             get

diff --git a/win/C#/Controls/x264Panel.cs b/win/C#/Controls/x264Panel.cs
index 4b943e32f..a3bfd7a56 100644
--- a/win/C#/Controls/x264Panel.cs
+++ b/win/C#/Controls/x264Panel.cs
@@ -353,6 +353,9 @@ namespace Handbrake.Controls
         /// <summary>

         /// This function will update the X264 Query when one of the GUI widgets changes.

         /// </summary>

+        /// <param name="sender">

+        /// The sender.

+        /// </param>

         private void OnX264WidgetChange(string sender)

         {

             Animate(sender);

@@ -376,8 +379,8 @@ namespace Handbrake.Controls
         /// Called when the current x264 option string contains multiple (or a single) item(s) in it seperated by :

         /// it updates the current option that the widget corrosponds to, if it is already in thes string.

         /// </summary>

-        /// <param name="currentOptString"></param>

-        /// <param name="optNameToChange"></param>

+        /// <param name="currentOptString">The Current Option String</param>

+        /// <param name="optNameToChange">Name of the option to change</param>

         private void HasOptions(string currentOptString, string optNameToChange)

         {

             string thisOpt; // The separated option such as "bframes=3"

@@ -440,8 +443,8 @@ namespace Handbrake.Controls
                                 thisOpt = string.Empty;

                             else

                             {

-                                double psyrd = slider_psyrd.Value*0.1;

-                                double psytre = slider_psytrellis.Value*0.1;

+                                double psyrd = slider_psyrd.Value * 0.1;

+                                double psytre = slider_psytrellis.Value * 0.1;

 

                                 string rd = psyrd.ToString("f1");

                                 string rt = psytre.ToString("f1");

@@ -592,7 +595,7 @@ namespace Handbrake.Controls
         /// Handles 2 cases.  1 Where rtf_x264Query.Text is empty, and one where there is an option with no value,

         /// e.g no-fast-pskip

         /// </summary>

-        /// <param name="optNameToChange"></param>

+        /// <param name="optNameToChange">The Option Name to Change</param>

         private void HasNoOptions(IEquatable<string> optNameToChange)

         {

             string colon = string.Empty;

@@ -705,8 +708,8 @@ namespace Handbrake.Controls
                     query += string.Empty;

                 else

                 {

-                    double psyrd = slider_psyrd.Value*0.1;

-                    double psytre = slider_psytrellis.Value*0.1;

+                    double psyrd = slider_psyrd.Value * 0.1;

+                    double psytre = slider_psytrellis.Value * 0.1;

 

                     string rd = psyrd.ToString("f1");

                     string rt = psytre.ToString("f1");

@@ -780,17 +783,17 @@ namespace Handbrake.Controls
         /// <summary>

         /// Shows and hides controls based on the values of other controls.

         /// </summary>

-        /// <param name="sender"></param>

+        /// <param name="sender">The Sender</param>

         private void Animate(string sender)

         {

             /* Lots of situations to cover.

- 	           - B-frames (when 0 turn of b-frame specific stuff, when < 2 disable b-pyramid)

- 	           - CABAC (when 0 turn off trellis)

- 	           - analysis (if none, turn off 8x8dct)

- 	           - refs (under 2, disable mixed-refs)

- 	           - subme (if under 6, turn off psy-rd and psy-trel)

- 	           - trellis (if 0, turn off psy-trel)

- 	        */

+               - B-frames (when 0 turn of b-frame specific stuff, when < 2 disable b-pyramid)

+               - CABAC (when 0 turn off trellis)

+               - analysis (if none, turn off 8x8dct)

+               - refs (under 2, disable mixed-refs)

+               - subme (if under 6, turn off psy-rd and psy-trel)

+               - trellis (if 0, turn off psy-trel)

+             */

 

             switch (sender)

             {

@@ -893,7 +896,6 @@ namespace Handbrake.Controls
                         slider_psyrd.Value = 10;

                         lbl_psyrd.Visible = false;

 

-

                         slider_psytrellis.Visible = false;

                         slider_psytrellis.Value = 0;

                         lbl_psytrellis.Visible = false;

@@ -931,7 +933,6 @@ namespace Handbrake.Controls
             }

         }

 

-

         private void widgetControlChanged(object sender, EventArgs e)

         {

             Control changedControlName = (Control) sender;