sub: create sub_bitmap array even when using libass

One sub_bitmaps struct could contain either a libass ASS_Image list, or
a mplayer native list of sub-bitmaps. This caused code duplication in
vo_vdpau.c and bitmap_packer.c.

Avoid this by creating such a sub_bitmap array even with libass. This
basically copies the list and recreates it in mplayer's native format.
It gets rid of the code duplication, and will make implementing extended
subtitle and OSD rendering in other VOs easier.

Also do some cosmetic changes and other preparations for the following
commits.

1 files changed, 0 insertions, 2 deletions

diff --git a/mplayer.c b/mplayer.c
index d17be7c357..5c08cdf330 100644
--- a/mplayer.c
+++ b/mplayer.c
@@ -2638,8 +2638,6 @@ static int redraw_osd(struct MPContext *mpctx)
 {
     struct sh_video *sh_video = mpctx->sh_video;
     struct vf_instance *vf = sh_video->vfilter;
-    if (sh_video->output_flags & VFCAP_OSD_FILTER)
-        return -1;
     if (vo_redraw_frame(mpctx->video_out) < 0)
         return -1;
     mpctx->osd->sub_pts = mpctx->video_pts;