[MPlayer-dev-eng] [PATCH] Dynamic list of EOSD sources

Nicolas George nicolas.george at normalesup.org
Sun Sep 12 21:11:23 CEST 2010


Le sextidi 26 fructidor, an CCXVIII, Reimar Döffinger a écrit :
> Why do you think that? PPC64 e.g. has a lot more issues with unsigned
> than with int. Also since overflow is undefined for signed, the compiler
> can do more optimizations.

I was rather thinking of divisions versus shifts: x / 2 can be coded as x >>
1 only if the compiler can be certain x is positive. But you have probably
more experience about various architectures than me.

> Haven't looked if you did, but just make sure that is documented, then
> it's no problem.

I did.

Updated version of the patch:

- the rest of the unsigned -> int;
- adapted to the new indentation of vf_ass.c;
- an additional Doxygen comment;
- sizeof(*ptr) rather than sizeof(type).

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-eosd-dynamic-20100912-2105.diff
Type: text/x-diff
Size: 22277 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100912/22e9100b/attachment-0001.diff>
-------------- 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/20100912/22e9100b/attachment-0001.pgp>


More information about the MPlayer-dev-eng mailing list