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



commit 889023a20561a3b8d3d423e7771d3d7ce6ada335
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                     |   11 +++++++++++
 glom/import_csv/csv_parser.cc |    2 --
 2 files changed, 11 insertions(+), 2 deletions(-)
---
diff --git a/ChangeLog b/ChangeLog
index 8a3527b..7d05acb 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-11  Murray Cumming  <murrayc murrayc com>
 
 	List view: Make the retry option actually work afer entering invalid data.
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]