[MPlayer-dev-eng] [PATCH] av_sub: support multiple rectangles.

[Nicolas George]

Le quartidi 4 floréal, an CCXX, Reimar Döffinger a écrit :
> What I meant to say is that I think it would be good to have
> the size 0 case commented here, too, not just at the other place.
> Because otherwise one might wonder why you wouldn't just return NULL
> in that case.

Ok.

> Maybe add a comment that clearing alpha is not enough
> because we need the brightness value to always be <= alpha?
> Just because I myself had forgotten that again.

Ok.

> IMO you don't need to send another patch, just commit.

Thanks, pushed.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120422/3bd73a44/attachment.asc>