Adam Jackson <[email protected]> writes:

>> +        if (name != BAD_RESOURCE) {
>> +            err = RRConfigureOutputProperty(output->randr_output, name,
>> +                                            FALSE, TRUE, TRUE,
>> +                                            2, prop_range);
>
> There's no good reason to make this a range, is there?

Well, it's a fixed integer for each output, which can either be
considered an enumeration which can have only one value or a value in a
range. I picked 'range' because it didn't seem like this could be
considered an enumeration to me.

If you like the other option better, that'd be fine with me :-)

-- 
-keith

Attachment: signature.asc
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to