makedepend: Changes to 'master'

Alan Coopersmith alanc at kemper.freedesktop.org
Sun May 13 04:14:57 UTC 2018


 ifparser.c |    6 +++---
 main.c     |    3 ---
 pr.c       |   53 +++++++++++++++++++++++++++++++++++++++++++++--------
 3 files changed, 48 insertions(+), 14 deletions(-)

New commits:
commit d15117c98a071b8fd30fb163fc45cb37db760654
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 5 10:54:59 2018 -0700

    Simplify writing of output lines to Makefile
    
    Instead of writing everything to a temporary buffer, and then using
    fwrite() to have it fputc() one character at a time into the stdio
    buffer, just use fprintf() directly to save a copy and write in
    larger blocks.
    
    Testing on Solaris on makedepend's own source files showed a
    reduction in memcpy's from 4037 to 3108, and in _dowrite calls
    in stdio from 1173 to 168, but no change in actual write calls
    from stdio's buffer to the file.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 4f410f3ed29fdb0f98be1e01930ba1a909a36ef0
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 5 10:05:44 2018 -0700

    Remove unused variable whitespace
    
    Use of it was removed in commit 275c17136006f9d, but we kept setting
    it anyway.
    
    Resolves gcc 7.3 warning:
    main.c: In function ‘getnextline’:
    main.c:587:10: warning: variable ‘whitespace’ set but not used [-Wunused-but-set-variable]
      boolean whitespace = FALSE;
              ^~~~~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 223b315a9148ec0091837f2e369bbb3ec5d49e11
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 5 10:00:38 2018 -0700

    Fix bad indentation in ifparser.c
    
    ifparser.c: In function ‘parse_product’:
    ifparser.c:300:5: warning: this ‘else’ clause does not guard... [-Wmisleading-indentation]
         else
         ^~~~
    ifparser.c:302:2: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘else’
      break;
      ^~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 13385d8add69156805f824cedcdad2986a23662d
Author: Antonio Larrosa <alarrosa at suse.com>
Date:   Tue Feb 2 18:31:27 2016 +0100

    Quote colons in filenames/paths
    
    Makefile doesn't like colons in filenames/paths so they must
    be quoted in the output. Otherwise makedepend doesn't work with
    full paths that contain a colon.
    
    V2: Use quoted filename when measuring name length
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>



More information about the xorg-commit mailing list