> On 08 Jun 2015, at 15:38, Marc Mutz <[email protected]> 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
_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to