coapp/coapp — pull request #64 by voltagex (2012-08-23 06:55:18+00:00)
Title: Silenced some warnings. Replaces #63

I've cleaned up a few things.
Remaining warnings are:

Warning	4	Unreachable code detected	coapp\CLI\CoAppMain.cs	320	25	Cli
Normal warning about a return before a break statement.

Warning	5	The variable 'completedThisPackage' is assigned but its value is never used	coapp\Packaging\Client\PackageManager.cs	337	17	CoApp.Client
Visual Studio doesn't seem to notice the use of a variable in an embedded statement

Warning	1	Bitwise-or operator used on a sign-extended operand; consider casting to a smaller unsigned type first	coapp\Toolkit\Win32\Architecture.cs	457	52	CoApp.Cleaner
Warning	2	Bitwise-or operator used on a sign-extended operand; consider casting to a smaller unsigned type first	coapp\Toolkit\Win32\Architecture.cs	457	52	CoApp.Service
Warning	3	Bitwise-or operator used on a sign-extended operand; consider casting to a smaller unsigned type first	coapp\Toolkit\Win32\Architecture.cs	457	52	CoApp.Toolkit
Lalalalalalala native code I can't hear you!

EventLog.EventID seems to be deprecated and I've replaced it with InstanceID as per http://msdn.microsoft.com/en-us/library/system.diagnostics.eventlogentry.instanceid(v=vs.100)
Please check this one.

Apologies for the duplicated pull requests, I didn't realise that branches are *very* important when sending requests.