xserver: Branch 'master'

Keith Packard keithp at kemper.freedesktop.org
Sat Mar 29 00:03:50 PDT 2014


 mi/miinitext.c |    8 --------
 1 file changed, 8 deletions(-)

New commits:
commit 8468e24430988058b150dc1daef3aad6ba9fa698
Author: Adam Jackson <ajax at redhat.com>
Date:   Thu Mar 27 11:48:39 2014 -0400

    mi: Hush extension initialization (#75870)
    
    Printing these as ErrorF is fairly obnoxious, since it means the
    non-hardware servers now spew stuff to the console for entirely routine
    events.  And actually, printing these at all is fairly obnoxious, since
    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.
    
    Signed-off-by: Adam Jackson <ajax at redhat.com>
    Reviewed-by: Daniel Stone <daniel at fooishbar.org>
    Signed-off-by: Keith Packard <keithp at keithp.com>

diff --git a/mi/miinitext.c b/mi/miinitext.c
index 5b45ab4..1d90516 100644
--- a/mi/miinitext.c
+++ b/mi/miinitext.c
@@ -365,7 +365,6 @@ void
 LoadExtensionList(const ExtensionModule ext[], int size, Bool builtin)
 {
     ExtensionModule *newext;
-    const char *msg;
     int i;
 
     /* Make sure built-in extensions get added to the list before those
@@ -375,14 +374,7 @@ LoadExtensionList(const ExtensionModule ext[], int size, Bool builtin)
     if (!(newext = NewExtensionModuleList(size)))
         return;
 
-    if (builtin)
-        msg = "Initializing built-in";
-    else
-        msg = "Loading";
-
     for (i = 0; i < size; i++, newext++) {
-        ErrorF("%s extension %s\n", msg, ext[i].name);
-
         newext->name = ext[i].name;
         newext->initFunc = ext[i].initFunc;
         newext->disablePtr = ext[i].disablePtr;


More information about the xorg-commit mailing list