[Bug 613081] info: make GST_PTR_FORMAT etc. work on win32, OSX and non-glibc systems

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=613081
  GStreamer | gstreamer (core) | git

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.1.1

--- Comment #15 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-04-13 10:19:33 UTC ---
Well, let's try this and see what happens:


commit 0e1dd050a56dcbfbd9583fc4f43ed3c3adc1a5f2
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Fri Apr 12 23:58:52 2013 +0100

    printf: don't build if debugging subsystem was disabled

commit 5299a0cd65e3233dd61a5c8862519be4961f1b27
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Wed Apr 10 11:51:37 2013 +0100

    printf: deal with some of the HAVE_FOO used in the printf code

    Probably needs some more work for MSVC.

commit 3c1d9c6d41e4f209af5be1baa933234d6ef51560
Author: Tim-Philipp Müller <tim.muller at collabora.co.uk>
Date:   Mon Apr 8 19:42:27 2013 +0100

    printf: fix alloca use for windows with mingw32

    Don't use just GLIB_HAVE_ALLOCA_H to check if alloca is available,
    that's just for the header. GLib may define alloca for us otherwise
    too irrespective of GLIB_HAVE_ALLOCA_H.

    Fixes compiler warning with mingw32:
    gst/printf/vasnprintf.c:73:0: warning: "alloca" redefined

commit ff292d530c126191bf220f70a8d114aa8f54e4b9
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sun Apr 7 20:11:21 2013 +0100

    printf: enable and fix compiler warnings

    But suppress -Wformat-nonliteral warnings since sprintf
    is used with a runtime-generated format string in our
    vasnprintf implementation.

commit 97b3948a9fd86da7f80b0703060a0a3324f899e4
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sun Apr 7 18:21:00 2013 +0100

    printf: fix up dodgy use of #if HAVE_FOO and #if !HAVE_FOO

    Should use #ifdef and #ifndef.

commit 79d6b91e27fd187cdaad78728c91a59104a3c85d
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sun Apr 7 17:36:29 2013 +0100

    printf: mark internal functions as internal

commit 3778c1878cf59b486228fcf3c500a091e7607e71
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sun Apr 7 17:29:02 2013 +0100

    printf: skip pointer extension signifier chars after %p

    So they don't get printed after the serialised pointer string.

commit 6d8a6470d59142ff1de3841894bb33e71d133566
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sun Apr 7 17:21:10 2013 +0100

    printf: don't leak serialised pointer extension strings

commit 9a9b449c5f1e872b85b2888754fa41b5b4b271a2
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sun Apr 7 17:02:55 2013 +0100

    printf: handle old GST_PTR_FORMAT %P and GST_SEGMENT_FORMAT %Q defines too

    For binary backwards compatibility.

commit 5803da553ca354362f3673503988745b7eb60607
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sun Apr 7 16:41:40 2013 +0100

    printf: make printf parser recognise our pointer extension format

    and call the hook to get a string for the pointer instead.

    https://bugzilla.gnome.org/show_bug.cgi?id=613081

commit fe7f7135e0399bd5a970164a5ac962c6800a69bd
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sat Mar 30 18:28:38 2013 +0000

    printf: add infrastructure for pointer extensions hook

    Does not do anything yet. On a sidenote, we can't just use
    %p\001 or so to signal the extension because g-i complains
    about an invalid ascii character then, so have to resort to
    something more elaborate, such as %p\aA etc.

    https://bugzilla.gnome.org/show_bug.cgi?id=613081

commit 7b1994428074a05795d7fe4418e29f921c2c0a09
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sat Mar 30 17:20:13 2013 +0000

    info: use new internal printf for debug message printing

    and remove all the printf extension/specifier stuff for
    the system printf. Next we need to add back the custom
    specifiers to our own printf implementation.

    https://bugzilla.gnome.org/show_bug.cgi?id=613081

commit 8fc876f09fb4f76902cedcb2edb650fad9f76021
Author: Tim-Philipp Müller <tim at centricular.net>
Date:   Sat Mar 30 15:13:32 2013 +0000

    printf: add our own printf implementation for debug logging

    We will add support for our own printf modifiers, so we can
    get nice debug log output on all operating systems irrespective
    of the specific libc version used.

    https://bugzilla.gnome.org/show_bug.cgi?id=613081

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.