[FFmpeg-devel] Reduce the size of AVRtpPayloadTypes

Michael Niedermayer michaelni
Thu Mar 27 14:16:37 CET 2008


On Thu, Mar 27, 2008 at 01:44:15PM +0100, Luca Abeni wrote:
[...]
> > changing it to enc_name[6] would be
> > more effective, especially on 64 bit systems (probably no relevant
> > difference on 32 bit systems though, except maybe better cache
> > utilization which I doubt is relevant here).
> 
> Is this the preferred ffmpeg's style?
> If yes, I'll change it in "enc_name[6]" (I think in this case the two
> solutions are equivalent, and this is just a matter of taste).

The pointers make the table unshareable with PIC (ironic ...)
They also all require a relocation, even with PIC (also ironic ...)

And the pointer means more overhead and an extra indirection.
So there are many arguments against this.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080327/3f0a2c57/attachment.pgp>



More information about the ffmpeg-devel mailing list