Class names should be CamelCase
Unified indenting and other whitespace issues
Deal now has performPlay

Massive changes to Play -- perform obviously moved out, some functions moved out
of .hpp, Trick now actually takes ownership of the cards it gets (they were actually leaking
earlier!) and therefore is not copyable anymore. Play now keeps a history in the
form of a list of Tricks. The equality operator for Suit and Denomination seems
to be dangerous, but I only moved it, so no changes there.
One more change to Deal -- bidding and play are now unique pointers and are
initiated when the proper function is called. This is not only neccessary for
Play, but will also allow for calling update() when implementing Observable properly.
Removed some now unnecessary includes in Play
Added one include back, turned out to be necessary
Observable stuff now actually calls update()

Also -- Observable got a virtual destructor, to get rid of clang++ extra warnings.
This makes sense.
Merge branch 'master' of https://github.com/mike1729/BridgeTCS1

Conflicts:
	src/Arbiter.hpp
	src/Hand.hpp