Re: [PATCH 0/2] MR5923: winegstreamer: Use affinity to determine 'max-threads' value.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/wg_parser.c:
>  pthread_cond_signal(&parser->init_cond);
>
> }
>
> +static void deep_element_added_cb(GstBin *self, GstBin *sub_bin, GstElement *element, gpointer user)
> +{
> - GstElementFactory *factory = NULL;
> - const char *name = NULL;
> -
> - if (element)
> -
>    factory = gst_element_get_factory(element);
> -
> - if (factory)
> -
>    name = gst_element_factory_get_longname(factory);
> -
> - if (name && strstr(name, "libav H.264"))

Do we need to check the name? Could we just set the property unconditionally? If we need to check the name I would rather check that the element short name starts with `avdec_`, as is the case for every libav decoder.