[MPlayer-dev-eng] [PATCH] vf_delogo: variable rectangle

Nicolas George nicolas.george at normalesup.org
Mon May 23 18:34:29 CEST 2011


Le tridi 3 prairial, an CCXIX, Reimar Döffinger a écrit :
> Yes, I'd say go ahead.

Done.

> But I'd like to say that despite the criticism I do appreciate your
> efforts to improve things.
> For example I also do agree that the struct_realloc function is useful
> and the current name and location is really crappy.
> Though I also think the most straight-forward way to get things moving
> (and in the process fully understand the requirements and issues) is
> to just extract common code with as little changes as possible as a
> first step.

No problem. I am still not happy to see unchecked mallocs and duplicated
code all over the place, but I guess you are too. And you convinced me that
the way I was trying to proceed is not suitable in this situation.

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/20110523/126f249d/attachment.asc>


More information about the MPlayer-dev-eng mailing list