Correct usage of boost test value printing

Looking at the [documentation][1] it seems that the intended method of
printing "custom" types is implementing the << operator.

This commit replaces the apparently platform-specific usage of
boost::test_tools::print_log_value with operator << implementation.

[1]: http://www.boost.org/doc/libs/1_60_0/libs/test/doc/html/boost_test/test_output/testing_tool_output_disable.html

Author: dannyedel

testing/testhelpers.cc

@@ -14,3 +14,7 @@ QString TestHelpers::pdfFilename(const std::string& basename) {
 std::ostream& operator << (std::ostream& where, const QSize& what) {
 	return where << "QSize(" << what.width() << 'x' << what.height() << ')';
 }
+
+std::ostream& operator << (std::ostream& where, const QColor& what) {
+	return where << "QColor(" << what.name().toStdString() << ")";
+}

testing/testhelpers.hh

@@ -56,17 +56,5 @@ namespace TestHelpers {
 /** Print a QSize to a standard output stream */
 std::ostream& operator << (std::ostream& where, const QSize& what);
 
-/** Print QRgb values as #123456 html-style-strings within boost test
- * log messages */
-namespace boost{
-	namespace test_tools{
-		namespace tt_detail {
-			template<>
-			inline
-			void
-			print_log_value<QColor>::operator()(std::ostream& where, const QColor& what) {
-				where << what.name().toStdString();
-			}
-		}
-	}
-}
+/** Print a QColor to a standard output stream */
+std::ostream& operator << (std::ostream& where, const QColor& what);