[glom/glom-1-14] CSV Import: Actually preview and import the field values.
- From: Murray Cumming <murrayc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [glom/glom-1-14] CSV Import: Actually preview and import the field values.
- Date: Fri, 12 Nov 2010 10:45:37 +0000 (UTC)
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]