[Development] Asking for a FF exception for ICU based QStringConverter

[Thiago Macieira]

On Thursday, 9 June 2022 10:16:19 PDT Alexander Akulich wrote:
> Hi everyone,
> 
> On Wed, Jun 8, 2022 at 5:18 PM Fabian Kosmale <fabian.kosmale at qt.io> wrote:
> > What remains to be done:
> > - There are concerns that mixing Qt versions might lead to an unbounded
> > memory leak with the current implementation.
> I can't find any precedent but if we *really* want then probably we
> can get the "compiled with" Qt version from QCoreApplication
> (QCoreApplicationPrivate::app_compile_version).

Doesn't work for libraries.

> 1. Allocate the resources only if the version >= 6.4.0
> 2. Probably add a "TODO: remove me" in 6.x.0 (e.g. after the next LTS)

Much simpler to add an overloaded constructor with an extra, dummy member. If 
you recompile your sources, you get the new one with the new codecs. If you 
don't recompile, you don't get the new codecs and thus can't leak.

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Cloud Software Architect - Intel DCAI Cloud Engineering