gst: fix awkward dest_format inout parameter in query utility functions
authorTim-Philipp Müller <tim.muller@collabora.co.uk>
Tue, 26 Jul 2011 23:17:02 +0000 (00:17 +0100)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Thu, 28 Jul 2011 10:23:15 +0000 (11:23 +0100)
commitef8ca3549e85df93664179117729ef9b982919ea
treefd6723387b59d3ca1e98b789f14a6402a4681448
parentcf18fa9ce580ec60cd51911e6db0aa8915d45883
gst: fix awkward dest_format inout parameter in query utility functions

The idea was originally that if one passed &dest_fmt with
dest_fmt=GST_FORMAT_DEFAULT, then the code answering the query
could change dest_fmt to the actual default format used. However,
in more than half a decade of GStreamer 0.10 no piece of code in
GStreamer has ever used that feature, nor are there that many
users of this API that actually check whether the format returned
is the original format passed before using the values returned.
Also, it's just annoying-to-use API in its own right.

For all these reasons, make it so that the destination format is
passed directly and can't be changed by the element queried.
docs/random/porting-to-0.11.txt
gst/gstquery.c
gst/gstutils.c
gst/gstutils.h