Ok.  This is the last one for today.

Here is the bug

https://bugs.freedesktop.org/show_bug.cgi?id=37622

and here is the commit I'd like to cherry-pick to the -3-4 branch

http://cgit.freedesktop.org/libreoffice/calc/commit/?id=099fd0fd91485797362259e45b1a971dda0511a3

The old code incorrectly returned NULL when the last font ID was passed,
because of the wrong boundary check.

In XLS, font IDs are zero-based when it's less than 4, but it becomes
one based when greater than 4 according to the spec.  This lookup
behavior was accidentally altered during ScfDefList removal.

Review & sign-off appreciated.

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
<[email protected]>

_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to