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

Adam Jackson ajax at nwnk.net
Wed Feb 21 16:42:09 UTC 2018


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.

Merged 1-5, thanks:

remote: I: patch #205455 updated using rev 1310241e8d606687811c7394811fe337c3e4de11.
remote: I: patch #205456 updated using rev 7fc757986947ad89d76fc0fd3d69f5fdeefc9055.
remote: I: patch #205457 updated using rev edc00e020310d8f53ebc16295c373fe84ec6ec87.
remote: I: patch #205458 updated using rev 089a1b0f9fa2e80457dc3d5f05c84565c9113a21.
remote: I: patch #205459 updated using rev 9237c5e2878c088a6cd26ea380763465a9e67012.
remote: I: 5 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/xorg/xserver
   a05453266..9237c5e28  master -> master

- ajax


More information about the xorg-devel mailing list