More code style changes

Revisions based on some of the guidelines listed in
http://blogs.msdn.com/b/brada/archive/2005/01/26/361363.aspx and
http://msdn.microsoft.com/en-us/library/ms229043.aspx (possibly more to
come along these lines).

No functionality changes, just renaming private members / parameters /
local variables, and rearranging code.

Author: Dave Wyatt

Commands/AddLogFileCommand.cs

@@ -0,0 +1,81 @@
+
+// ReSharper disable MemberCanBePrivate.Global
+// ReSharper disable UnusedAutoPropertyAccessor.Global
+// ReSharper disable UnusedMember.Global
+
+namespace PSLogging.Commands
+{
+    using System.Management.Automation;
+    
+    [Cmdlet(VerbsCommon.Add, "LogFile")]
+    public class AddLogFileCommand : PSCmdlet
+    {
+        private ScriptBlock errorCallback;
+        private LogFile inputObject;
+        private string path;
+        private StreamType streams = StreamType.All;
+
+        #region Parameters
+
+        [Parameter(ParameterSetName = "AttachExisting",
+            Mandatory = true,
+            Position = 0,
+            ValueFromPipeline = true)]
+        public LogFile InputObject
+        {
+            get { return this.inputObject; }
+            set { this.inputObject = value; }
+        }
+
+        [Parameter(ParameterSetName = "New")]
+        public ScriptBlock OnError
+        {
+            get { return this.errorCallback; }
+            set { this.errorCallback = value; }
+        }
+
+        [Parameter(Mandatory = true,
+            Position = 0,
+            ParameterSetName = "New")]
+        public string Path
+        {
+            get { return this.path; }
+            set
+            {
+                this.path = System.IO.Path.IsPathRooted(value)
+                                ? value
+                                : System.IO.Path.Combine(this.SessionState.Path.CurrentLocation.Path, value);
+            }
+        }
+
+        [Parameter(ParameterSetName = "New")]
+        public StreamType StreamType
+        {
+            get { return this.streams; }
+            set { this.streams = value; }
+        }
+
+        #endregion
+
+        protected override void EndProcessing()
+        {
+            LogFile logFile;
+
+            if (this.ParameterSetName == "New")
+            {
+                logFile = new LogFile(this.path, this.streams, this.errorCallback);
+                this.WriteObject(logFile);
+            }
+            else
+            {
+                logFile = this.inputObject;
+            }
+
+            HostIoInterceptor.GetInterceptor().AddSubscriber(logFile);
+        }
+    } // End AddLogFileCommand class
+}
+
+// ReSharper restore MemberCanBePrivate.Global
+// ReSharper restore UnusedAutoPropertyAccessor.Global
+// ReSharper restore UnusedMember.Global

Commands/AddOutputSubscriberCommand.cs

@@ -0,0 +1,93 @@
+// ReSharper disable MemberCanBePrivate.Global
+// ReSharper disable UnusedAutoPropertyAccessor.Global
+// ReSharper disable UnusedMember.Global
+
+namespace PSLogging.Commands
+{
+    using System.Management.Automation;
+
+    [Cmdlet(VerbsCommon.Add, "OutputSubscriber")]
+    public class AddOutputSubscriberCommand : PSCmdlet
+    {
+        private ScriptBlockOutputSubscriber inputObject;
+        private ScriptBlock onWriteDebug;
+        private ScriptBlock onWriteError;
+        private ScriptBlock onWriteOutput;
+        private ScriptBlock onWriteVerbose;
+        private ScriptBlock onWriteWarning;
+
+        #region Parameters
+
+        [Parameter(ParameterSetName = "AttachExisting",
+            Mandatory = true,
+            ValueFromPipeline = true,
+            Position = 0)]
+        public ScriptBlockOutputSubscriber InputObject
+        {
+            get { return this.inputObject; }
+            set { this.inputObject = value; }
+        }
+
+        [Parameter(ParameterSetName = "New")]
+        public ScriptBlock OnWriteDebug
+        {
+            get { return this.onWriteDebug; }
+            set { this.onWriteDebug = value; }
+        }
+
+        [Parameter(ParameterSetName = "New")]
+        public ScriptBlock OnWriteError
+        {
+            get { return this.onWriteError; }
+            set { this.onWriteError = value; }
+        }
+
+        [Parameter(ParameterSetName = "New")]
+        public ScriptBlock OnWriteOutput
+        {
+            get { return this.onWriteOutput; }
+            set { this.onWriteOutput = value; }
+        }
+
+        [Parameter(ParameterSetName = "New")]
+        public ScriptBlock OnWriteVerbose
+        {
+            get { return this.onWriteVerbose; }
+            set { this.onWriteVerbose = value; }
+        }
+
+        [Parameter(ParameterSetName = "New")]
+        public ScriptBlock OnWriteWarning
+        {
+            get { return this.onWriteWarning; }
+            set { this.onWriteWarning = value; }
+        }
+
+        #endregion
+
+        protected override void EndProcessing()
+        {
+            ScriptBlockOutputSubscriber subscriber;
+
+            if (this.ParameterSetName == "New")
+            {
+                subscriber = new ScriptBlockOutputSubscriber(this.onWriteOutput,
+                                                             this.onWriteDebug,
+                                                             this.onWriteVerbose,
+                                                             this.onWriteError,
+                                                             this.onWriteWarning);
+                this.WriteObject(subscriber);
+            }
+            else
+            {
+                subscriber = this.inputObject;
+            }
+
+            HostIoInterceptor.GetInterceptor().AddSubscriber(subscriber);
+        }
+    } // End AddOutputSubscriberCommand class
+}
+
+// ReSharper restore MemberCanBePrivate.Global
+// ReSharper restore UnusedAutoPropertyAccessor.Global
+// ReSharper restore UnusedMember.Global