Hi Troy,
Many thanks for your nice patch: you found out the right place to hack!
However I have some concerns here...
On Thu, 2011-04-14 at 18:04 +1000, Troy Rollo wrote:
> + if ( nLen > 0 && aPrevStr.GetChar(nLen - 1) == '.' &&
This is quite restrictive as it only clears the ending '.' I think that
the same should apply whatever the suffix is.. Would one want to have
[1-] as the reference?
You could either hack the MakeNumString itself or get inspired of it to
get the suffix to remove.
> + ( aRefNumStr.Len() == 0 ||
> + aRefNumStr.GetChar(0) == '(' )
Why that test for strings starting with '(' ?
Don't hesitate to ask for help if you need some to improve the patch and
keep doing nice work like this one.
Regards,
--
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
