*Cracks tester knuckles* *Crashes program 10 different ways* *Decides to read code instead*  catch (Exception) is generally considered extremely bad style, since it catches program-fatal exceptions, as well. I have used it myself, as well, though. I sometimes wish there were a  catch (!SystemException), but the .Net developers have decided that the likes of SystemException and ApplicationException were a mistake... I find it very useful but meh. ASSS. Although, they were not consistent at all, so it is a bad idea to rely on these two anyway. Woah, this list item got too long.   I have having garbage comments - XML comments like <param name="sender">sender</param>. But meh, many coders are too sloppy to remove that. A lot actually want those there. Freaks     File IO has almost no exception handling. Every read, every open, every write can throw a mul!@#$%^&*ude of exceptions very easily! At least do the lazy catch (Exception)     The type column is a bit skinny - lots of things get cut off at default width for me.  Your non-XML commenting is pretty good!