Remove usage of unsavoury language in comments

Logs/src/logging.cpp

@@ -194,7 +194,7 @@ namespace Gambit
        //std::cout<<"Sorting tags..."<<std::endl;
        for(std::set<int>::iterator tag = mail.tags.begin(); tag != mail.tags.end(); ++tag)
        {
-         // Debugging crap... to be deleted.
+         // Debugging stuff... to be deleted.
          // std::cout<<"Sorting tag "<<tag2str()[*tag]<<std::endl;
          // std::cout<<"empty?"<<components().empty()<<std::endl;
          // for(std::set<int>::iterator tag2 = components().begin(); tag2 != components().end(); ++tag2)

Printers/include/gambit/Printers/printers/hdf5printer/hdf5tools.hpp

@@ -6,7 +6,7 @@
 ///  HDF5 databases.
 ///
 ///  Currently I am using the C++ bindings for
-///  HDF5, however they are a bit crap and it may
+///  HDF5, however they not great and it may
 ///  be better to just write our own.
 ///
 ///  *********************************************

Printers/src/printers/hdf5printer/hdf5tools.cpp

@@ -6,7 +6,7 @@
 ///  HDF5 databases.
 ///
 ///  Currently I am using the C++ bindings for
-///  HDF5, however they are a bit crap and it may
+///  HDF5, however they not great and it may
 ///  be better to just write our own.
 ///
 ///  *********************************************

ScannerBit/src/scanners/twalk/twalk.cpp

@@ -477,7 +477,7 @@ namespace Gambit
                 set_resume_params.dump(); // Better way
                 // This works I think, but it still has problems. In particular,
                 // it looks like you must resume with the same number of processes
-                // that you started the run with, which is kind of crap.
+                // that you started the run with, which is not good.
             }
 
             for (auto &&gd : gDev) delete gd;