Fixed the headers of nunit-console for the 2.9.7 release

Pretty simple change to prevent confusion.
Just so we don't go around changing code we would not otherwise be changing
I'm OK.

My observation as an itinerant coach is that companies generally
over-standardize in their style guidelines, perhaps underestimating the
adaptability of their programmers. :-)


On Fri, Aug 8, 2014 at 1:30 PM, Rob Prouse <notifications@github.com> wrote:

> In src/nunit-console/Program.cs:
>
> > @@ -173,8 +172,8 @@ private static void WriteHeader()
> >              string versionText = executingAssembly.GetName().Version.ToString(3);
> >
> >              string programName = "NUnit Console Runner";
> > -            string copyrightText = "Copyright (C) 2011 Charlie Poole.\r\nAll Rights Reserved.";
> > -            string configText = "";
> > +            string copyrightText = "Copyright (C) 2014 Charlie Poole.\r\nAll Rights Reserved.";
> > +            string configText = String.Empty;
>
> As you say, there is no difference and there isn't a common style
> guideline that I know of, but it is the style where I work so it has become
> habit and I think it makes you more likely to use other Empty calls like
> Guid.Empty or EventArgs.Empty. Pretty minor if you prefer "" I am fine
> with either.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nunit/nunit-console/pull/37/files#r16014558>.
>