Crash when using KIdentityManagement::IdentityManager

Ingo Klöcker kloecker at kde.org
Fri Apr 16 13:07:54 BST 2021


On Freitag, 16. April 2021 12:20:21 CEST David Faure wrote:
> On jeudi 15 avril 2021 22:59:40 CEST Volker Krause wrote:
> > That's two different things, a very long inline method will most likely
> > result in a single local/non exported symbol in the consuming binary with
> > regular local function calls to it, rather than being copy/pasted into
> > each
> > call site. In that sense it's no different from any other non-exported
> > method in that library then.
> 
> Ah, so it's copy/pasted once per "consumer target" (library or executable),
> not (necessarily) for each call site, I see!
> 
> > > > > In any case, pimpl'ing kills this idea, so I suggest we make the
> > > > > rule
> > > > > simple: no QVector/QList iterators in public API.
> 
> I now added this to the "You cannot..." section of
> https://community.kde.org/Policies/Binary_Compatibility_Issues_With_C%2B%2B#
> The_Do.27s_and_Don.27ts

And, I guess, we should take the KF6 opportunity and replace (deprecate now, 
remove in KF6) the problematic API, e.g. with the change suggested by Volker.

Regards,
Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20210416/ec9ba864/attachment.sig>


More information about the kde-pim mailing list