[Development] Qt 5.5.0 header diff: QtMultimedia.diff
Lopes Yoann
Yoann.Lopes at theqtcompany.com
Mon Jun 8 15:17:26 CEST 2015
> On 08 Jun 2015, at 15:38, Marc Mutz <marc.mutz at kdab.com> wrote:
>
> On Friday 05 June 2015 10:10:52 Frederik Gladhorn wrote:
>> + typedef QPair<qreal, qreal> FrameRateRange;
>
> Should be a struct, not a QPair:
>
> FrameRateChange chg = ...;
> chg.first... // what's first???
> chg.second... // what's second???
It’s "…Range", not “…Change" ;)
I think having ‘first’ and ‘second’ for a range is pretty self-explanatory.
—
Yoann
More information about the Development
mailing list