On Mon, Feb 12, 2018 at 10:18:16AM +0000, Jean-Marc Lasgouttes wrote:

> This seems simple enough, but of course should be checked by Mac/Win people.

+1

> I also removed traces of it in the English User's Guide, and added some
> entries in release notes. Feel free to improve on the wording.

Thank you, looks good.

> Comments welcome.

Why do you leave in the following code?

 #if defined(Q_WS_X11) || defined(QPA_XCB)
        pixmapCacheCB->setEnabled(false);
 #endif

I'm guessing it's because you want to make as minimal changes as
possible?

> PS: I do not think it is worth applying this to master. We will either end
> up fixing the feature or removing it altogether.

+1

Scott

Attachment: signature.asc
Description: PGP signature

Reply via email to