lmi-commits
[Top][All Lists]
Advanced

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

[lmi-commits] [lmi] master 0eec56e 5/5: Regularize "TODO" markers


From: Greg Chicares
Subject: [lmi-commits] [lmi] master 0eec56e 5/5: Regularize "TODO" markers
Date: Tue, 30 Jan 2018 19:47:37 -0500 (EST)

branch: master
commit 0eec56ecf207156eb1faa308eb36f35b0411c27e
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Regularize "TODO" markers
    
    Generally, lmi uses "TODO" only in "TODO ??", which marks a defect.
    Replaced several instances that lacked "??" and didn't seem like serious
    defects with milder, more specific "PDF !!" markers.
---
 html.cpp               | 2 +-
 interpolate_string.cpp | 2 +-
 wx_table_generator.cpp | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/html.cpp b/html.cpp
index b08d80b..8f545d2 100644
--- a/html.cpp
+++ b/html.cpp
@@ -62,7 +62,7 @@ std::string attribute::as_string() const
     if(!value_.empty())
         {
         s += "=";
-        // TODO: Escape quotes.
+        // PDF !! Escape quotes.
         s += value_;
         }
     return s;
diff --git a/interpolate_string.cpp b/interpolate_string.cpp
index a372230..841f33e 100644
--- a/interpolate_string.cpp
+++ b/interpolate_string.cpp
@@ -43,7 +43,7 @@ struct section_info
 
     // Name of the section, i.e. the part after "#".
     //
-    // TODO: In C++14 this could be replaced with string_view which would
+    // PDF !! In C++14 this could be replaced with string_view which would
     // save on memory allocations without compromising safety, as we know
     // that the input string doesn't change during this function execution.
     std::string const name_;
diff --git a/wx_table_generator.cpp b/wx_table_generator.cpp
index 8ebe75d..22ccfaa 100644
--- a/wx_table_generator.cpp
+++ b/wx_table_generator.cpp
@@ -130,7 +130,7 @@ void wx_table_generator::do_output_horz_separator(int x1, 
int x2, int y)
 
 void wx_table_generator::do_output_vert_separator(int x, int y1, int y2)
 {
-    // TODO: add a possibility to have a thick border between the columns.
+    // PDF !! add a possibility to have a thick border between the columns.
     dc_.DrawLine(x, y1, x, y2);
 }
 



reply via email to

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