lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [5923] Resolve several marked defects in obsolescent code


From: Greg Chicares
Subject: [lmi-commits] [5923] Resolve several marked defects in obsolescent code
Date: Wed, 09 Jul 2014 12:32:45 +0000

Revision: 5923
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=5923
Author:   chicares
Date:     2014-07-09 12:32:44 +0000 (Wed, 09 Jul 2014)
Log Message:
-----------
Resolve several marked defects in obsolescent code

Modified Paths:
--------------
    lmi/trunk/ihs_server7702io.cpp

Modified: lmi/trunk/ihs_server7702io.cpp
===================================================================
--- lmi/trunk/ihs_server7702io.cpp      2014-06-27 17:03:11 UTC (rev 5922)
+++ lmi/trunk/ihs_server7702io.cpp      2014-07-09 12:32:44 UTC (rev 5923)
@@ -38,7 +38,8 @@
 #include <sstream>
 #include <string>
 
-// TODO ?? NEED DECISION on i/o representation.
+// NEED DECISION on i/o representation. (New code writes as many
+// decimal digits as are significant.)
 //#include <limits>
 namespace
 {
@@ -50,7 +51,8 @@
 // 123456789 1 0 0 0 10000 0 .02 0 1000000 1000000 0 0 0 1000000
 // 45 45 0 0 0 0 0 0 1000000 1000000 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0
 
-// TODO ?? It would be better to perform range checking on bool input.
+// It would be better to perform range checking on bool input. (New
+// code accomplishes that by using enumerative types.)
 
 //============================================================================
 std::istream& operator>> (std::istream& is, Server7702Input& z)
@@ -108,7 +110,8 @@
     // I give the names with a suffix "0" here,
     // but they need to be repeated with "1", "2",
     // and so on for as many flats as permitted.
-    // TODO ?? NEED DECISION how many is that? --assume it's 3
+    // NEED DECISION how many is that? --assume it's 3 (New code more
+    // flexibly uses input sequences.)
     is >> z.NewPermanentFlatAmount0;
     is >> z.OldPermanentFlatAmount0;
     is >> z.NewPermanentFlatAmount1;
@@ -359,8 +362,8 @@
 //      << setfill('0')
 //  os.width(prec + 1 + prec);
 //  os.fill('0');
-// TODO ?? NEED DECISION Do they want everything aligned?
-// TODO ?? NEED DECISION If so, what width for each item?
+// NEED DECISION Do they want everything aligned? (New code uses xml.)
+// NEED DECISION If so, what width for each item? (New code uses xml.)
 
     os << std::setiosflags(std::ios_base::fixed) << std::setprecision(prec);
 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]