[gstreamer-bugs] [Bug 320765] ffmpegcolorspace win32+msvc compliant

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=320765
 GStreamer | gst-plugins-base | Ver: HEAD CVS





------- Comment #4 from Sebastien Moutte  2006-01-31 12:36 UTC -------
ok, so 
#define GST_FF_VID_CAPS_NEW(mimetype, ...)                      \
    (context != NULL) ?                                         \
    gst_caps_new_simple (mimetype,                              \
        "width",     G_TYPE_INT,   context->width,              \
        "height",    G_TYPE_INT,   context->height,             \
        "framerate", GST_TYPE_FRACTION,                         \
        (gint) context->frame_rate, (gint) context->frame_rate_base, \
        __VA_ARGS__, NULL)                                      \
    :                                                           \
    gst_caps_new_simple (mimetype,                              \
        "width",     GST_TYPE_INT_RANGE, 1, G_MAXINT,           \
        "height",    GST_TYPE_INT_RANGE, 1, G_MAXINT,           \
        "framerate", GST_TYPE_FRACTION_RANGE, 0, 1, G_MAXINT, 1,\
        __VA_ARGS__, NULL)

becomes something like 
void gst_ff_vid_caps_new(AVCodecContext * context, const char *mimetype, ...)
{
  GstCaps *caps = NULL;
  va_list var_args;

  if(context != NULL) {
    caps = gst_caps_new_simple (mimetype,
      "width",     G_TYPE_INT,   context->width,
      "height",    G_TYPE_INT,   context->height,
      "framerate", GST_TYPE_FRACTION,
      (gint) context->frame_rate, (gint) context->frame_rate_base,
      NULL);
  }
  else {
    caps = gst_caps_new_simple (mimetype,
      "width",     GST_TYPE_INT_RANGE, 1, G_MAXINT,
      "height",    GST_TYPE_INT_RANGE, 1, G_MAXINT,
      "framerate", GST_TYPE_FRACTION_RANGE, 0, 1, G_MAXINT, 1,
      NULL);
  }

  va_start (var_args, mimetype);
  gst_caps_set_simple_valist (caps, ????, var_args);
  va_end (var_args);

  return caps;
}

What should be first field name ? can we extract it from var_args or use NULL
then it should take the first one?


-- 
Configure bugmail: http://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.