[PATCH xserver 08/10] xf86-video-modesetting: Create CONNECTOR_ID properties for outputs

Keith Packard keithp at keithp.com
Tue Jan 23 00:32:25 UTC 2018


Adam Jackson <ajax at nwnk.net> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.x.org/archives/xorg-devel/attachments/20180123/439813e5/attachment.sig>


More information about the xorg-devel mailing list