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

List:       kde-devel
Subject:    Compile error fix patch of lilo-config (CVS of 24/02)
From:       Toshitaka Fujioka <toshitaka () kde ! gr ! jp>
Date:       2001-02-25 9:46:33
[Download RAW message or body]

Hello,

The following error was displayed when I compiled lilo-config:

---------------Error message ----------------------------
String.cc: In method `StringList::operator String() const':
String.cc:365: ambiguous overload for `String != const char[2]'
String.cc:365: candidates are: operator !=(char *, char *) <builtin>
/usr/include/g++-2/std/bastring.h:508:                 operator !=<char, \
string_char_traits<char>, alloc>(const \
basic_string<char,string_char_traits<char>,__default_alloc_template<true,0> > &, \
const char *) String.cc:365: ambiguous overload for `String != const char[2]'
String.cc:365: candidates are: operator !=(char *, char *) <builtin>
/usr/include/g++-2/std/bastring.h:508:                 operator !=<char, \
string_char_traits<char>, alloc>(const \
basic_string<char,string_char_traits<char>,__default_alloc_template<true,0> > &, \
const char *) make[3]: *** [String.lo] Error 1
--------------------------------------------------------

I made the patch which fixed this error.

I attached a patch. Please review. :-)
If my patch is wrong, please advise.


["kdeadmin-lilo-config-CompileError-fix-20010225.diff" (text/plain)]

diff -ur kdeadmin.org/lilo-config/common/String.cc kdeadmin/lilo-config/common/String.cc
--- kdeadmin.org/lilo-config/common/String.cc	Sun Feb 25 14:14:29 2001
+++ kdeadmin/lilo-config/common/String.cc	Sun Feb 25 14:15:35 2001
@@ -362,7 +362,7 @@
 	String s="";
 	for(const_iterator it=begin(); it!=end(); it++) {
 		s+=(*it);
-		if(s.right()!="\n" && s.right()!="\r")
+		if(s.right()!='\n' && s.right()!='\r')
 			s+="\n";
 	}
 	return s;

>> Visit http://master.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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

Configure | About | News | Add a list | Sponsored by KoreLogic