[PATCH xserver 6/6] mi: Mention extension loading in verbose logs

Emil Velikov emil.l.velikov at gmail.com
Wed Feb 21 17:21:09 UTC 2018


On 21 February 2018 at 16:42, Adam Jackson <ajax at nwnk.net> wrote:
> On Mon, 2018-02-19 at 15:18 +0000, Emil Velikov wrote:
>> From: Emil Velikov <emil.velikov at collabora.com>
>>
>> Listing the extensions is useful, despite being annoying for normal
>> usecases. Print it only when extra (lvl 3) vebose is requested.
>
> I appreciate lowering the log verbosity, but this repeats both of the
> problems I mentioned when I took this printf out the first time: "a)
> we're printing a line for every extension, whether it's enabled or
> not, and b) we're not actually initializing the extension at this
> point." (8468e2443)
>
> Put it just before the call to ->initFunc in InitExtensions to fix both
> issues.
>
Ack. Patch updated [1] with a copy in your inbox, in case it helps.


> Merged 1-5, thanks:
>
Thanks
Emil

[1] https://patchwork.freedesktop.org/patch/205981/


More information about the xorg-devel mailing list