ScSetStringParam::mbSetTextCellFormat (was: [REVIEW 3-6, 3-6-0] [PUSHED 3-6] resolved fdo#52205 do not force all text cells in CSV import)

2012-07-20 Thread Eike Rathke
Hi Kohei, On Friday, 2012-07-20 09:25:54 -0400, Kohei Yoshida wrote: > The old behavior for ScSetStringParam::mbSetTextCellFormat was > somewhat confusing and there was a reason for that. I now slightly > changed the behavior of that flag and you commit kept that new > behavior. I inspected pla

Re: [REVIEW 3-6, 3-6-0] [PUSHED 3-6] resolved fdo#52205 do not force all text cells in CSV import

2012-07-20 Thread Markus Mohrhard
Hey, 2012/7/20 Kohei Yoshida : > On 07/19/2012 12:31 PM, Eike Rathke wrote: >> >> Hi, >> >> Please review and cherry-pick to 3-6 and 3-6-0 >> >> http://cgit.freedesktop.org/libreoffice/core/commit/?id=b7dbc768a71ccfb567e3b2979e57d0d1318977cf >> that fixes https://bugs.freedesktop.org/show_bug.cgi?

Re: [REVIEW 3-6, 3-6-0] [PUSHED 3-6] resolved fdo#52205 do not force all text cells in CSV import

2012-07-20 Thread Kohei Yoshida
On 07/19/2012 12:31 PM, Eike Rathke wrote: Hi, Please review and cherry-pick to 3-6 and 3-6-0 http://cgit.freedesktop.org/libreoffice/core/commit/?id=b7dbc768a71ccfb567e3b2979e57d0d1318977cf that fixes https://bugs.freedesktop.org/show_bug.cgi?id=52205 I reviewed and tested several csv files a