Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index


Hi,

It's me again.

I just fixed the following bug:
https://bugs.freedesktop.org/show_bug.cgi?id=32570

and would like to port the fix to 3.3.  Here is the relevant commit
http://cgit.freedesktop.org/libreoffice/calc/commit/?id=98764831e3b6c02d7630c61a6c389ce4318787bd

Here is some risk analysis.

The worst case this change would cause is that some formula strings that
should have been flagged bad may be accepted (e.g. 100.A1 where the 100
is a sheet name).

But I would say the risk of that happening is quite low since we already
automatically detect numerical sheet names and quote them, so chances
are that we won't run into an unquoted numerical sheet name followed by
a '.'.  Plus, the current code breaks legitimate cases as evident in the
bug report.

Also, we support multiple formula syntaxes and a '.' is not always the
sheet-to-reference separator, which makes this check totally wrong in
our code base.

A review and a sign-off would be great.

Kohei

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


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.