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

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; ]

Regards,
Muthu Subramanian

On Mon, 2010-12-13 at 16:28 -0500, Kohei Yoshida wrote:
> Hi there,
>
> I'd like to port a patch to the libreoffice-3-3 branch to ignore
> preceding and trailing spaces in cells during csv import.  The change is
> already committed on master, and the attached is the total difference
> against the current libreoffice-3-3 branch.  The change on master
> consists of several commits.
>
> With this change, a csv file such as this
>
> 30, 40, 50
>
> or
>
> 30 , 40, 50
>
> will be imported as three number cells.  Currently, Calc imports number
> cells as strings in presence of spaces before or after the number.
>
> This change is only applicable when the user leaves the "Detect special
> number" option *un*-checked, which is off by default.
>
> Patch reviews are appreciated.
>
> Kohei
>
> _______________________________________________
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice