[FFmpeg-devel] [PATCH] Make the inet_aton replacement endian independent

Michael Niedermayer michaelni
Fri Jan 8 03:30:46 CET 2010


On Thu, Jan 07, 2010 at 06:21:42PM +0200, Martin Storsj? wrote:
> On Thu, 7 Jan 2010, Ronald S. Bultje wrote:
> 
> > On Fri, Jan 1, 2010 at 11:04 AM, Martin Storsj? <martin at martin.st> wrote:
> > > Currently, the replacement which is used if inet_aton isn't available,
> > > only works correctly on little-endian. The attached patch makes it endian
> > > independent.
> > 
> > -    add->s_addr=(add4<<24)+(add3<<16)+(add2<<8)+add1;
> > +    add->s_addr = htonl((add1<<24) + (add2<<16) + (add3<<8) + add4);
> > 
> > I'd add spaces: add3_<<_8 (around all) while you're at it. Otherwise
> > patch OK, if tested.
> 
> Yes, tested on both big- and little-endian.
> 
> > Do you have a SVN account or do I need to apply for you?
> 
> I don't have any SVN account, so feel free to apply with any formatting 
> changes you like.

If ronald thinks you should have a svn account then i dont mind if you
get one (assuming you agree to our development policy)

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

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- 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/20100108/a30f5fa4/attachment.pgp>



More information about the ffmpeg-devel mailing list