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


On 01/17/2014 12:28 PM, Matúš Kukan wrote:
commit 72b5343bd16deec540d8cd148cd7aebd74e92c16
Author: Matúš Kukan <matus.kukan@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

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.