[PATCH weston 01/15] libweston: improve weston_output_disable() comments

Daniel Stone daniel at fooishbar.org
Tue Apr 4 20:47:11 UTC 2017


Hi,

On 4 April 2017 at 20:22, Armin Krezović <krezovic.armin at gmail.com> wrote:
> On 04.04.2017 12:58, Pekka Paalanen wrote:
>> Reorder some paragraphs to be more logically ordered. Rewrite the
>> description of the backend-specific disable function to explain the
>> semantics instead of the mechanics. Remove the paragraph about
>> pending_output_list as unnecessary details.
>>
>> Add a big fat comment on why we call output->disable() always instead of
>> only for actually enabled outputs.
>>
>
> Might also note that it's only relevant to drm-backend at the moment.

Even then, the atomic series makes it ignore ->disable() for
non-enabled-at-startup outputs, so we can disable them in the same
request as we configure the others.

Cheers,
Daniel


More information about the wayland-devel mailing list