[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: [PATCH] Re: KSocks on non-linux systems.
From:       George Staikos <staikos () kde ! org>
Date:       2001-03-19 21:30:45
[Download RAW message or body]

On Monday 19 March 2001 15:16, Rob Napier wrote:

> Ah crude. Doesn't quite work. The problem is on Solaris if XPG4 isn't
> defined, then this needs to be int instead of size_t (which is uint,
> so it probably "matters"). If I just set it to 'int', though, is this
> going to break other platforms? Any AIX testers?
>
> HPUX11 defines socklen_t only if _XOPEN_SOURCE_EXTENDED is set. Is
> this set for HPUX builds? I'm betting no. Otherwise these functions
> all use 'int' (just like Solaris).
>
> For the moment, I recommend changing "size_t" to "int" (corrected
> patch below) and see if that breaks anyone. The downside is that you
> won't really know whether you're having trouble unless it breaks at
> runtime, since it only throws a warning about signness being screwed
> up.

   I don't have time for any coding over the next 3-4 days.  Please apply any 
patches as you see fit.  I'll change anything later if there's functional 
problems.

-- 

George Staikos

[prev in list] [next in list] [prev in thread] [next in thread]