noone checks the return value of ReadCsvLine

tools/inc/tools/stream.hxx

@@ -464,8 +464,7 @@ public:
                     field may also be escaped with a preceding backslash.
                     Normally, quotes are escaped by doubling them.
 
-                    @return
-                    sal_True if no stream error.
+                    check Stream::good() to detect IO problems during read
 
                     @ATTENTION
                     Note that the string returned may be truncated even inside
@@ -483,7 +482,7 @@ public:
                     characters will be spoiled, and a subsequent ReadCsvLine()
                     may start under false preconditions.
                   */
-    sal_Bool        ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
+    void            ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
                         const String& rFieldSeparators, sal_Unicode cFieldQuote,
                         sal_Bool bAllowBackslashEscape = sal_False);
 

tools/source/stream/stream.cxx

@@ -1037,7 +1037,7 @@ inline const sal_Unicode* lcl_UnicodeStrChr( const sal_Unicode* pStr,
     return 0;
 }
 
-sal_Bool SvStream::ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
+void SvStream::ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
         const String& rFieldSeparators, sal_Unicode cFieldQuote,
         sal_Bool bAllowBackslashEscape)
 {
@@ -1102,7 +1102,6 @@ sal_Bool SvStream::ReadCsvLine( String& rStr, sal_Bool bEmbeddedLineBreak,
             }
         }
     }
-    return nError == SVSTREAM_OK;
 }
 
 /*************************************************************************