mirror of
https://github.com/mpv-player/mpv.git
synced 2025-12-28 05:33:14 +00:00
sub: change how libass output is converted to RGBA in some cases
This affects VOs (or other code which render OSD) which does not support the LIBASS format, but only RGBA. Instead of having a converter stage in osd.c, make mp_ass_packer_pack() output directly in RGBA. In general, this is work towards refcounted subtitle images. Although we could keep the "converter" design, doing it this way seems simpler, at least considering the current situation with only 2 OSD formats. It also prevents copying & packing the data twice, which will lead to better performance. (Although I guess this case is not important at all.) It also fixes --force-rgba-osd-rendering when used with vo_opengl, vo_vdpau, and vo_direct3d.
This commit is contained in:
24
sub/osd.c
24
sub/osd.c
@@ -127,8 +127,6 @@ struct osd_state *osd_create(struct mpv_global *global)
|
||||
.text = talloc_strdup(obj, ""),
|
||||
.progbar_state = {.type = -1},
|
||||
};
|
||||
for (int i = 0; i < OSD_CONV_CACHE_MAX; i++)
|
||||
obj->cache[i] = talloc_steal(obj, osd_conv_cache_new());
|
||||
osd->objs[n] = obj;
|
||||
}
|
||||
|
||||
@@ -251,11 +249,6 @@ static void render_object(struct osd_state *osd, struct osd_object *obj,
|
||||
if (!sub_formats[format] || opts->force_rgba_osd)
|
||||
format = SUBBITMAP_RGBA;
|
||||
|
||||
bool formats[SUBBITMAP_COUNT];
|
||||
memcpy(formats, sub_formats, sizeof(formats));
|
||||
if (opts->force_rgba_osd)
|
||||
formats[SUBBITMAP_LIBASS] = false;
|
||||
|
||||
*out_imgs = (struct sub_bitmaps) {0};
|
||||
|
||||
check_obj_resize(osd, res, obj);
|
||||
@@ -285,25 +278,8 @@ static void render_object(struct osd_state *osd, struct osd_object *obj,
|
||||
if (out_imgs->num_parts == 0)
|
||||
return;
|
||||
|
||||
if (obj->cached.change_id == obj->vo_change_id && formats[obj->cached.format])
|
||||
{
|
||||
*out_imgs = obj->cached;
|
||||
return;
|
||||
}
|
||||
|
||||
out_imgs->render_index = obj->type;
|
||||
out_imgs->change_id = obj->vo_change_id;
|
||||
|
||||
if (formats[out_imgs->format])
|
||||
return;
|
||||
|
||||
bool cached = false; // do we have a copy of all the image data?
|
||||
|
||||
if (formats[SUBBITMAP_RGBA] && out_imgs->format == SUBBITMAP_LIBASS)
|
||||
cached |= osd_conv_ass_to_rgba(obj->cache[3], out_imgs);
|
||||
|
||||
if (cached)
|
||||
obj->cached = *out_imgs;
|
||||
}
|
||||
|
||||
// draw_flags is a bit field of OSD_DRAW_* constants
|
||||
|
||||
Reference in New Issue
Block a user