[Libreoffice] Patch for ignoring preceding and trailing spaces in cell during csv import

Kohei Yoshida kyoshida at novell.com
Tue Dec 14 05:21:52 PST 2010


On Tue, 2010-12-14 at 15:19 +0530, Muthu Subramanian K wrote:
> Hi Kohei,
> 
> Is there a specific reason why functions like:
> rStr.EraseLeadingAndTrailingChars() weren't used on the string?
> [ then something like n=rStr.getLength() if(!n) return false; ]

If we do that we'd be parsing the same character sequence twice in each
number cell, which will make the import slower, especially with large
csv files.

So, this is the reason why I avoid it.

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
<kyoshida at novell.com>



More information about the LibreOffice mailing list