Removed the bool cast operator from FBasicString. It turns out it was causing

way, way, way more problems than it solved.  A number of libbu++ tests were
inacurate because of it, there were problems in several other programs, and
there may be more that have problems we haven't found yet because of this.

This will most likely cause complitaion errors, especially in places we didn't
expect, where strings were being stored into or passed as integers and the like.

In cases where you were just testing a string, just call the "isSet()" function,
which is functionally equivellent to the old bool cast operator.

1 files changed, 3 insertions, 0 deletions

diff --git a/src/csvreader.cpp b/src/csvreader.cpp
index b045773..c63fe06 100644
--- a/src/csvreader.cpp
+++ b/src/csvreader.cpp
@@ -71,6 +71,9 @@ Bu::FString Bu::CsvReader::decodeExcel( Bu::FString::iterator &i )
 
         for(; i && (*i == ' ' || *i == '\t'); i++ ) { }
         
+        if( !i )
+                return sRet;
+
         if( *i == '\"' )
         {
                 for( i++ ; i; i++ )