On 21 February 2018 at 16:42, Adam Jackson <a...@nwnk.net> wrote:
> On Mon, 2018-02-19 at 15:18 +0000, Emil Velikov wrote:
>> From: Emil Velikov <emil.veli...@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/
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to