[glom/glom-1-14] CSV Import: Actually preview and import the field values.



commit ff748442edada70b50eeb20b5ff6709ec7955087
Author: Murray Cumming <murrayc murrayc com>
Date:   Fri Nov 12 11:32:44 2010 +0100

    CSV Import: Actually preview and import the field values.
    
    * glom/import_csv/csv_parser.cc: advance_field(): Remove an excess continue,
    so that we actually add the character to the field's text. This error
    was probably introduced while removing the non-exceptions ifdefs.
    
    We really must fix those import unit tests so that something like this does
    not happen again.

 ChangeLog                     |   13 ++++++++++++-
 glom/import_csv/csv_parser.cc |    2 --
 2 files changed, 12 insertions(+), 3 deletions(-)
---
diff --git a/ChangeLog b/ChangeLog
index 488e252..95c71ec 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2010-11-12  Murray Cumming  <murrayc murrayc com>
+
+	CSV Import: Actually preview and import the field values.
+
+	* glom/import_csv/csv_parser.cc: advance_field(): Remove an excess continue, 
+	so that we actually add the character to the field's text. This error 
+	was probably introduced while removing the non-exceptions ifdefs.
+	
+	We really must fix those import unit tests so that something like this does
+	not happen again.
+
 2010-11-04  Murray Cumming  <murrayc murrayc com>
 
 	Remove the gconfmm dependency, because we don't use it.
@@ -19959,7 +19970,7 @@ There is also a glom 1.0 branch for maintenance.
 
 	* Updated for changed gtkmm2 pack_start() and TreeView APIs.
 
-2002-03-19  Murray Cumming  <murrayc usa net
+2002-03-19  Murray Cumming  <murrayc usa net>
 
 	* Used pkg-config checks instead of the various old .m4 macros,
 	and changed Xerces C++ version to 1.7.
diff --git a/glom/import_csv/csv_parser.cc b/glom/import_csv/csv_parser.cc
index 632fe59..2c90f85 100644
--- a/glom/import_csv/csv_parser.cc
+++ b/glom/import_csv/csv_parser.cc
@@ -247,8 +247,6 @@ Glib::ustring::const_iterator CsvParser::advance_field(const Glib::ustring::cons
       {
         break;
       }
-
-      continue;
     }
 
     field += c; // Just so that we don't need to iterate through the field again, since there is no Glib::ustring::substr(iter, iter)



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