perky@users.sourceforge.net
<perky@users.sourceforge.net> wrote:                  
                                                        
                                                      
                                                      
                             
> Update of /cvsroot/python/python/dist/src/Modules   
                                                      
                             
> In directory sc8-pr-cvs1:/tmp/cvs-serv3617          
                                                      
                             
>                                                     
                                                      
                             
> Modified Files:                                     
                                                      
                             
>       socketmodule.c                                
                                                      
                             
> Log Message:                                        
                                                      
                             
> Fix erroneus argument parsing of socket.htons() on
64bit big endian                                      
                               
> machines.                                           
                                                      
                             
                                                      
                                                      
                             
This is cool but not enough. There is another place
where 64 bit machines fail. Here is diff. With it my
OpenBSD/sparc64 passes all socket tests (test_socket).
Of course OpenBSD/i386 is still working OK.           
                                                      
              
                                                      
                                                      
                             
[diff attached]

Hye-Shik Chang had already replied. Here is what he said:

[...] But return value type of inet_addr(3) and       
                                                      
            its size has complex issues because it is
used for not only OpenBSD but                         
                                           also BeOS,
IRIX, old BSDs, Windows and etc. that I don't have an
access.                                               
                   I think you'd better to submit it to
SF tracker [...]

So I'm submiting it.

Logged In: YES 
user_id=883833

Yuck. This looks ugly. Close this patch and I'll resubmit
it. Sorry :/

Logged In: YES 
user_id=21627

Ok, closed.