On 11/27/2011 08:57 AM, Daisuke Nishino wrote:
Exposing functioin pointers to outside is not a good practice I think.
This patch encapsulates them.
Also, since fnLeaveSelect is called from only one location, I eliminated
the indirection.

Great, thanks. Pushed (I simplified the "this->*this->" calls by removing the redundant second "this->").

I do not find your license statement at <http://wiki.documentfoundation.org/Development/Developers>, can you add it there (I assume you contribute under LGPLv3+/MPL 1.1+)?

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

Reply via email to