[Libreoffice-commits] .: patches/dev300

Kohei Yoshida kohei at kemper.freedesktop.org
Tue Nov 9 21:46:40 PST 2010


 patches/dev300/apply                                   |    4 -
 patches/dev300/stream-read-csv-always-single-line.diff |   52 -----------------
 2 files changed, 56 deletions(-)

New commits:
commit fa2e6f597d787292e3f93b493fee1734baa61bb7
Author: Kohei Yoshida <kyoshida at novell.com>
Date:   Wed Nov 10 00:45:36 2010 -0500

    Obsolete patch removed.  The correct fix is in the upstream repo.

diff --git a/patches/dev300/apply b/patches/dev300/apply
index 2f1df67..16ebdc4 100755
--- a/patches/dev300/apply
+++ b/patches/dev300/apply
@@ -2053,10 +2053,6 @@ experimental_ooapi.diff, flr
 # It's empty but please don't remove this section.
 SectionOwner => kohei
 
-# when reading csv file, don't read multiple physical lines for one logical
-# line even if matching quotes are on separate lines.
-stream-read-csv-always-single-line.diff, n#523517, kohei
-
 [ AutoLayout ]
 impress-autolayout.diff, cocofan
 
diff --git a/patches/dev300/stream-read-csv-always-single-line.diff b/patches/dev300/stream-read-csv-always-single-line.diff
deleted file mode 100644
index 63933ea..0000000
--- a/patches/dev300/stream-read-csv-always-single-line.diff
+++ /dev/null
@@ -1,52 +0,0 @@
-diff --git tools/inc/tools/stream.hxx tools/inc/tools/stream.hxx
-index a8f0874..b8334db 100644
---- tools/inc/tools/stream.hxx
-+++ tools/inc/tools/stream.hxx
-@@ -535,6 +535,13 @@ public:
-                     If sal_True, an embedded quote character inside a quoted
-                     field may also be escaped with a preceding backslash.
-                     Normally, quotes are escaped by doubling them.
-+                    
-+                    @param bAllowQuotedMultiLine
-+                    If true, a quoted sequence may have line breaks, which
-+                    means multiple lines may be read if the opening and
-+                    closing quotes are on different lines.  If false, then
-+                    only a single line is read even when the closing quote of
-+                    a quoted seqence is not reached.
- 
-                     @return
-                     sal_True if no stream error.
-@@ -557,7 +564,7 @@ public:
-                   */
-     sal_Bool        ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
-                         const String& rFieldSeparators, sal_Unicode cFieldQuote,
--                        sal_Bool bAllowBackslashEscape = sal_False);
-+                        sal_Bool bAllowBackslashEscape = sal_False, bool bAllowQuotedMultiLine = false);
- 
-     void			SetBufferSize( sal_uInt16 nBufSize );
-     sal_uInt16	GetBufferSize() const { return nBufSize; }
-diff --git tools/source/stream/stream.cxx tools/source/stream/stream.cxx
-index ae4dfff..bb9421f 100644
---- tools/source/stream/stream.cxx
-+++ tools/source/stream/stream.cxx
-@@ -1089,7 +1089,7 @@ inline const sal_Unicode* lcl_UnicodeStrChr( const sal_Unicode* pStr,
- 
- sal_Bool SvStream::ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
-         const String& rFieldSeparators, sal_Unicode cFieldQuote,
--        sal_Bool bAllowBackslashEscape)
-+        sal_Bool bAllowBackslashEscape, bool bAllowQuotedMultiLine)
- {
-     ReadUniOrByteStringLine( rStr);
- 
-@@ -1126,6 +1126,11 @@ sal_Bool SvStream::ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
-                 ++p;
-             }
- 
-+            if (!bAllowQuotedMultiLine)
-+                // Don't read more than a single line even if a quoted
-+                // sequence has not reached its closing quote.
-+                break;
-+
-             if (nQuotes % 2 == 0)
-                 break;
-             else


More information about the Libreoffice-commits mailing list