Hi,
On Thu, Jun 29, 2017 at 04:35:30PM +0200, Christophe Fergeau wrote:
> On Thu, Jun 29, 2017 at 01:58:07PM +0200, Victor Toso wrote:
> > Hi,
> >
> > On Thu, Jun 29, 2017 at 10:59:16AM +0200, Pavel Grunt wrote:
> > > On Wed, 2017-06-28 at 14:43 +0200, Victor Toso wrote:
> > > > From: Victor Toso <me@xxxxxxxxxxxxxx>
> > > >
> > > > The Playbin can provide the full pipeline which reduces the
> > > > overall maintenance in the code as we don't need to track which
> > > > decoder can work with our stream type.
> > > >
> > > > We need to maintain the GstCaps per SPICE_VIDEO_CODEC_TYPE in order
> > > > to
> > > > tell Playbin the type of data we expect. This much should be covered
> > > > by spice-protocol and very likely we will need to extend it in the
> > > > future to allow more settings that might not possible to verify at
> > > > runtime.
> > > >
> > > > This patch keeps previous code for compatibility reasons.
> > > >
> > > > Note that we have to wait Playbin to emit "source-setup" in order to
> > > > configure GstAppSrc with the capabilities of input stream. If in the
> > > > unlikely event of frames arriving while GstAppSrc is not setup, we
> > > > will drop those frames.
> > > >
> > > > Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
> > > > Signed-off-by: Victor Toso <me@xxxxxxxxxxxxxx>
> > > > ---
> > > > src/channel-display-gst.c | 99
> > > > ++++++++++++++++++++++++++++++++++++++++++++++-
> > > > 1 file changed, 97 insertions(+), 2 deletions(-)
> > > >
> > > > diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c
> > > > index 1b40002..df58de3 100644
> > > > --- a/src/channel-display-gst.c
> > > > +++ b/src/channel-display-gst.c
> > > > @@ -50,6 +50,9 @@ typedef struct SpiceGstDecoder {
> > > > guint timer_id;
> > > > } SpiceGstDecoder;
> > > >
> > > > +/* FIXME: With gstreamer version 1.9.0 and higher, we are using
> > > > playbin to
> > > > + * create the pipeline for us and for that reason we don't need to
> > > > keep track of
> > > > + * decoder's name anymore. */
> > > > static struct {
> > > > const gchar *dec_name;
> > > > const gchar *dec_caps;
> > > > @@ -83,6 +86,21 @@ G_STATIC_ASSERT(G_N_ELEMENTS(gst_opts) <=
> > > > SPICE_VIDEO_CODEC_TYPE_ENUM_END);
> > > > #define VALID_VIDEO_CODEC_TYPE(codec) \
> > > > (codec > 0 && codec < G_N_ELEMENTS(gst_opts))
> > > >
> > > > +typedef enum {
> > > > + GST_PLAY_FLAG_VIDEO = (1 << 0),
> > > > + GST_PLAY_FLAG_AUDIO = (1 << 1),
> > > > + GST_PLAY_FLAG_TEXT = (1 << 2),
> > > > + GST_PLAY_FLAG_VIS = (1 << 3),
> > > > + GST_PLAY_FLAG_SOFT_VOLUME = (1 << 4),
> > > > + GST_PLAY_FLAG_NATIVE_AUDIO = (1 << 5),
> > > > + GST_PLAY_FLAG_NATIVE_VIDEO = (1 << 6),
> > > > + GST_PLAY_FLAG_DOWNLOAD = (1 << 7),
> > > > + GST_PLAY_FLAG_BUFFERING = (1 << 8),
> > > > + GST_PLAY_FLAG_DEINTERLACE = (1 << 9),
> > > > + GST_PLAY_FLAG_SOFT_COLORBALANCE = (1 << 10),
> > > > + GST_PLAY_FLAG_FORCE_FILTERS = (1 << 11),
> > > > +} GstPlayFlags;
> > >
> > > It can cause problems if gstreamer exports, please give them a SPICE
> > > prefix
> >
> > "Thanks for the bug report, but we don't export plugin headers, sorry.
> >
> > You can copy and paste these into your code, or use API like
> > gst_util_set_object_arg() so you can use string nicks."
> >
> > So, it should never happen. Source:
> > https://bugzilla.gnome.org/show_bug.cgi?id=784279
>
> I'd add a bug reference before the enum so that people know why you
> copied and pasted this.
>
> Christophe
Sure, I've just added it.
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel