Changeset 4581

Show
Ignore:
Timestamp:
02/21/10 21:36:31 (5 months ago)
Author:
phk
Message:

Evict the [v]asprintf compat & configure stuff, we have stopped using
these functions.

The [v]asprintf() functions are convenient shorthand, but suffer from
a number of problems, portability, cornercases and performance being
the most relevant to Varnish.

Static buffers on the stack (see [v]bprintf macros, or sbufs (vsb)
are always better choices for us.

Location:
trunk/varnish-cache
Files:
4 removed
3 modified

Legend:

Unmodified
Added
Removed

  • trunk/varnish-cache/configure.ac

    r4558 r4581  
    211211# These functions are provided by libcompat on platforms where they 
    212212# are not available 
    213 AC_CHECK_FUNCS([asprintf vasprintf]) 
    214213AC_CHECK_FUNCS([setproctitle]) 
    215214AC_CHECK_FUNCS([srandomdev]) 

  • trunk/varnish-cache/include/Makefile.am

    r4469 r4581  
    1414        cli_priv.h \ 
    1515        cli_serve.h \ 
    16         compat/asprintf.h \ 
    1716        compat/daemon.h \ 
    1817        compat/execinfo.h \ 
    … …  
    2221        compat/strlcpy.h \ 
    2322        compat/strndup.h \ 
    24         compat/vasprintf.h \ 
    2523        flopen.h \ 
    2624        http_headers.h \ 

  • trunk/varnish-cache/lib/libvarnishcompat/Makefile.am

    r4056 r4581  
    88 
    99libvarnishcompat_la_SOURCES = \ 
    10         asprintf.c \ 
    1110        daemon.c \ 
    1211        execinfo.c \ 
    13         vasprintf.c \ 
    1412        setproctitle.c \ 
    1513        srandomdev.c \