Skip to content

Commit

Permalink
Remove the conversion from premultiplied alpha, ...
Browse files Browse the repository at this point in the history
...as I found a case where it makes the output almost unreadable, and
cannot find another test case where it makes it better, which I'm
pretty sure I had when I did this.

Signed-off-by: Rémi Denis-Courmont <[email protected]>
  • Loading branch information
Vincent Penquerc'h authored and Rémi Denis-Courmont committed Jan 23, 2010
1 parent 16f75b8 commit 6a89345
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions modules/codec/kate.c
Original file line number Diff line number Diff line change
Expand Up @@ -824,6 +824,7 @@ static void SubpictureReleaseRegions( subpicture_t *p_subpic )
}
}

#if 0
/*
* We get premultiplied alpha, but VLC doesn't expect this, so we demultiply
* alpha to avoid double multiply (and thus thinner text than we should)).
Expand Down Expand Up @@ -874,6 +875,7 @@ static void PostprocessTigerImage( plane_t *p_plane, unsigned int i_width )
}
PROFILE_STOP( tiger_renderer_postprocess );
}
#endif

/* Tiger renders can end up looking a bit crap since they get overlaid on top of
a subsampled YUV image, so there can be a fair amount of chroma bleeding.
Expand Down Expand Up @@ -975,7 +977,9 @@ static void TigerUpdateRegions( spu_t *p_spu, subpicture_t *p_subpic, const vide
}
PROFILE_STOP( tiger_renderer_render );

#if 0
PostprocessTigerImage( p_plane, fmt.i_width );
#endif
p_subpic->p_region = p_r;
p_sys->b_dirty = false;

Expand Down

0 comments on commit 6a89345

Please sign in to comment.