[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