[Development] Qt 5.5.0 header diff: QtMultimedia.diff

Thiago Macieira thiago.macieira at intel.com
Tue Jun 9 00:27:40 CEST 2015


On Tuesday 09 June 2015 00:12:47 Milian Wolff wrote:
> Sorry Thiago, but imo your answers are not on topic: The documentation is
> /wrong/. QList is _not_ the right container class to use "for most
> purposes". If, inside Qt, it is currently (ab)used for historical reasons,
> well then fine. But can be /please/ get rid of the prominent endorsement in
> the documentation? Saying it should be preferred when interacting with
> existing (Qt) API is something altogether different than saying it should
> be used by default. It is *not* usually faster than QVector, and that is
> /especially/ because of the way it stores items in memory. That it expands
> to less code is afaik true, but only to a small margin, according to the
> measurements done by e.g. Volker Krause on KF5 code.

Fair enough, go ahead and change it.

But Qt API that doesn't explicitly need an array for the data should continue 
to use QList.

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center




More information about the Development mailing list