[vlc-devel] [vlc-commits] vlc_memalign: use memalign() if posix_memalign() isn't here

Rafaël Carré rafael.carre at gmail.com
Thu Sep 22 21:30:10 CEST 2011


Le 22 septembre 2011 10:15, Rémi Denis-Courmont <remi at remlab.net> a écrit :
> Le jeudi 22 septembre 2011 05:31:57 Rafaël Carré, vous avez écrit :
>> vlc | branch: master | Rafaël Carré <funman at videolan.org> | Wed Sep 21
>> 21:25:17 2011 -0400| [0ad141d394dd86e930d1f27be2ea6ea7e698fddf] |
>> committer: Rafaël Carré
>>
>> vlc_memalign: use memalign() if posix_memalign() isn't here
>>
>> Also rename confusing vlc_free()
>
> Adding not just one but two relocations just because of some old version of
> some crappy OS is rather suboptimal...

Is that really a problem? afaict memalign() is not used in "hot" path

> Have fun maintaining the portability layer from now on.

You can revert if you want. What other possibility do we have:
posix_memalign() replacement?

We'd need to make vlc_memalign_free() part of libvlccore if we do that

-- 
Rafaël Carré



More information about the vlc-devel mailing list