On 01/17/2014 12:28 PM, Matúš Kukan wrote:
commit 72b5343bd16deec540d8cd148cd7aebd74e92c16
Author: Matúš Kukan <matus.ku...@collabora.com>
Date:   Thu Jan 9 20:18:49 2014 +0100

    fwk: Use constructor feature for ModuleUIConfigurationManager.

    And avoid css::uno::XInitialization protocol.

Removing XInitialization from a service implementation is always a delicate move regarding compatibility. Even though the new-style service css.ui.ModuleUIConfigurationManager does not mention XInitialization, the old-style variant (prior to <http://cgit.freedesktop.org/libreoffice/core/commit/?id=9c41a60c0d802c618ac414822615a5c9e9e92f9f> "fdo#46808, Convert ui::ModuleUIConfigurationManager service to new style") did, and there is always a chance that (external) code obtains the service implementation via a non-argument queryInstance call and then explicitly queries for XInitialization.

So although this change is most probably justified, it is nevertheless incompatible. Matúš, can you please manually list your relevant past changes at <https://wiki.documentfoundation.org/ReleaseNotes/4.3#API_changes> and add "[API CHANGE]" to the commit messages of relevant future changes (to allow for semi-automatic tracking)?

Thanks,
Stephan
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to