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

List:       kde-core-devel
Subject:    Re: Bug#20842: wrong string header for solaris
From:       Waldo Bastian <bastian () kde ! org>
Date:       2001-02-23 21:13:37
[Download RAW message or body]

On Friday 23 February 2001 12:55, you wrote:
> On Fri, Feb 23, 2001 at 12:49:23PM -0800, Waldo Bastian wrote:
> > > kio_man.cpp includes string.h for index() and rindex(). On Solaris,
> > > this should be strings.h. Sorry I don't have time right now to patch
> > > the configure scripts right now. A simple AC_CHECK_HEADERS like kdelibs
> > > uses) may be sufficient, but this might break AIX.
> > >
> > > Rob
> >
> > Which version of the files are we talking about? I don't see 'index()' or
> > 'rindex()' being used at all. What errors do you get? Please paste them.

Ah I see. The bug is in v1.29 of kio_man.cpp, but KDE 2.1 contains v1.28.
The solution is to replace the index/rindex with the strchr/strrchr POSIX 
calls.

Cheers,
Waldo
-- 
bastian@kde.org | SuSE Labs KDE Developer | bastian@suse.com

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