removing warnings.

diff --git a/nucleus/library/filesystem/filename_list.cpp b/nucleus/library/filesystem/filename_list.cpp
index 24d1522e6bf36b08c9ef8b659fa0144979167538..28ec738668ddde5bf799c06d28ef284033b30ff7 100644 (file)
--- a/nucleus/library/filesystem/filename_list.cpp
+++ b/nucleus/library/filesystem/filename_list.cpp
@@ -152,10 +152,9 @@ bool filename_list::member_with_state(const file_info &to_check, file_info::file
   return false;
 }
 
-astring filename_list::text_form() const
+astring filename_list::text_form(int max_lines) const
 {
   astring to_return;
-//hmmm: a length limit might be nice?
   for (int i = 0; i < elements(); i++) {
     to_return += a_sprintf("%d. ", i + 1);
     if (get(i))

diff --git a/nucleus/library/filesystem/filename_list.h b/nucleus/library/filesystem/filename_list.h
index f5834d79ce38d5c85090e0964549f1cf9c523f28..89c91c2d64ce2805a5ddd5ac4f0772415c305776 100644 (file)
--- a/nucleus/library/filesystem/filename_list.h
+++ b/nucleus/library/filesystem/filename_list.h
@@ -75,7 +75,8 @@ public:
     true if the size is different (when EQUAL_FILESIZE is true), when the timestamp is different
     (when EQUAL_TIMESTAMP is true), and when the checksum is different (you get the idea). */
 
-  basis::astring text_form() const;
+  //! max_lines is the maximum number of lines to print into the string.
+  basis::astring text_form(int max_lines = MAXINT32) const;
 
   virtual int packed_size() const;
 

diff --git a/nucleus/library/structures/hash_table.h b/nucleus/library/structures/hash_table.h
index 4eb0285620543af95183efef5737a5b1d49b11f3..2560bc7619a6f657233b5e39daa2ce6b43fca79b 100644 (file)
--- a/nucleus/library/structures/hash_table.h
+++ b/nucleus/library/structures/hash_table.h
@@ -59,7 +59,7 @@ public:
   The buckets are currently simple lists, but if the hashing algorithm is well
   chosen, then that's not a major problem.  This makes lookups a lot faster
   than a linear search, but no particular performance guarantees are made at
-  this time (hmmm).
+  this time.
 */
 
 template <class key_type, class contents>

diff --git a/nucleus/library/textual/xml_parser.h b/nucleus/library/textual/xml_parser.h
index 7b2011d8b26097af0bfbe8d7ce98c3f5bdfbc9de..fbc9653482cffd41d38b163a993a85c31ef2d9a4 100644 (file)
--- a/nucleus/library/textual/xml_parser.h
+++ b/nucleus/library/textual/xml_parser.h
@@ -25,9 +25,6 @@ namespace textual {
 
 //! Parses XML input and invokes a callback for the different syntactic pieces.
 
-// hmmm, could this be the first class ever named this?  perhaps it should be
-// in a textual namespace.  -->after current sprint.
-
 class xml_parser
 {
 public:

diff --git a/nucleus/tools/clam_tools/write_build_config.cpp b/nucleus/tools/clam_tools/write_build_config.cpp
index d3b9a812cad979196255538871c937366b474f36..f19d3999ecd757e6877fb81fa82dc10c1e9a0748 100644 (file)
--- a/nucleus/tools/clam_tools/write_build_config.cpp
+++ b/nucleus/tools/clam_tools/write_build_config.cpp
@@ -289,7 +289,7 @@ int write_build_config::execute()
   astring buffer;
   while (!ini.eof()) {
     int chars = ini.getline(buffer, MAX_LINE_SIZE);
-    if (!chars) continue;  // hmmm.
+    if (!chars) continue;  // hmmm: what does no chars mean?
     
     variable_tokenizer t;
     t.parse(buffer);

diff --git a/octopi/library/sockets/raw_socket.h b/octopi/library/sockets/raw_socket.h
index de98b310ad802248da06a5b4705474f5dfe55a0b..99e948981a44d0bc544a8fd011c648acca6139e8 100644 (file)
--- a/octopi/library/sockets/raw_socket.h
+++ b/octopi/library/sockets/raw_socket.h
@@ -61,9 +61,6 @@ public:
 
   DEFINE_CLASS_NAME("raw_socket");
 
-//  int open(
-//hmmm...
-
   int close(basis::un_int &socket);
     // disconnects, destroys and resets the "socket" to zero.