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

List:       kde-commits
Subject:    [Konversation] 85b2d8c: Remove LocaleString. It was causing crashes.
From:       Gary Cramblitt <garycramblitt () comcast ! net>
Date:       2010-07-01 13:27:52
Message-ID: 20100701132752.360E2BB5709 () projects ! kde ! org
[Download RAW message or body]

commit 85b2d8c478f5635408034f5215370e0c156fe78c
Author: Gary Cramblitt <garycramblitt@comcast.net>
Date:   Sat Aug 14 18:34:52 2004 +0000

    Remove LocaleString.  It was causing crashes.  Each component that displays a \
list of nicks will need to implement localeAware sorting.  
    svn path=/trunk/kdeextragear-2/konversation/; revision=337970

diff --git a/konversation/channelnick.h b/konversation/channelnick.h
index 46c3db5..d8e55e0 100644
--- a/konversation/channelnick.h
+++ b/konversation/channelnick.h
@@ -65,7 +65,7 @@ typedef KSharedPtr<ChannelNick> ChannelNickPtr;
 /** A ChannelNickMap is a list of ChannelNick pointers, indexed and sorted by
  *  lowercase nickname. 
  */
-typedef QMap<LocaleString,ChannelNickPtr> ChannelNickMap;
+typedef QMap<QString,ChannelNickPtr> ChannelNickMap;
 
 typedef QValueList<ChannelNickPtr> ChannelNickList;
 
@@ -73,7 +73,7 @@ typedef QValueList<ChannelNickPtr> ChannelNickList;
 /** A ChannelMembershipMap is a list of ChannelNickMap pointers, indexed and 
  *  sorted by lowercase channel name.
  */
-typedef QMap<LocaleString,ChannelNickMap *> ChannelMembershipMap;
+typedef QMap<QString,ChannelNickMap *> ChannelMembershipMap;
 
 
 #endif /* CHANNEL_NICK_H */
diff --git a/konversation/localestring.h b/konversation/localestring.h
deleted file mode 100644
index 73fcb22..0000000
--- a/konversation/localestring.h
+++ /dev/null
@@ -1,41 +0,0 @@
-/*
-  This program is free software; you can redistribute it and/or modify
-  it under the terms of the GNU General Public License as published by
-  the Free Software Foundation; either version 2 of the License, or
-  (at your option) any later version.
-*/
-
-/*
-  localestring.h  -  Class that sorts in localeAware order.
-  begin:     Wed Aug 04  2004
-  copyright: (C) 2002,2003,2004 by Dario Abatianni
-  email:     eisfuchs@tigress.com
-*/
-
-
-#ifndef LOCALE_STRING_H
-#define LOCALE_STRING_H
-
-/** A LocaleString is used as a key to a QMap.  Unlike QString, it sorts the QMap
-  * in localeAware order.
-  */
-class LocaleString : public QString
-{
-  public:
-    LocaleString() : QString() {};
-    LocaleString(const QString& s) : QString(s) {}
-    LocaleString(const LocaleString& s) : QString(s) {}
-    LocaleString& operator=(const QString& s) {
-      QString::operator=(s);
-      return *this; }
-    LocaleString& operator=(const LocaleString& s) {
-      QString::operator=(s);
-      return *this; }
-    inline bool operator<( const LocaleString &s1) { return (localeAwareCompare(s1) \
                < 0); }
-    inline bool operator<( const QString &s1) { return (localeAwareCompare(s1) < 0); \
                }
-    inline bool operator<( const char *s1) { return (localeAwareCompare(s1) < 0); }
-    inline bool operator<( QChar c) { return (localeAwareCompare(c) < 0); }
-    inline bool operator<( char ch) { return (localeAwareCompare(&ch) < 0); }
-};
-
-#endif
diff --git a/konversation/nickinfo.h b/konversation/nickinfo.h
index 3af3ba9..058a01b 100644
--- a/konversation/nickinfo.h
+++ b/konversation/nickinfo.h
@@ -25,7 +25,6 @@
 
 #include <kabc/addressbook.h>
 #include "addressbook.h"
-#include "localestring.h"
 
 class Server;
 
@@ -102,7 +101,7 @@ class NickInfo : public QObject, public KShared
 typedef KSharedPtr<NickInfo> NickInfoPtr;
 /** A NickInfoMap is a list of NickInfo objects, indexed and sorted by lowercase \
                nickname.
  */
-typedef QMap<LocaleString,NickInfoPtr> NickInfoMap;
+typedef QMap<QString,NickInfoPtr> NickInfoMap;
 
 #endif
 
diff --git a/konversation/nicksonline.cpp b/konversation/nicksonline.cpp
index e117ecc..308a9b8 100644
--- a/konversation/nicksonline.cpp
+++ b/konversation/nicksonline.cpp
@@ -205,7 +205,7 @@ void NicksOnline::updateServerOnlineList(Server* server, bool)
     while (child)
     {
       nextChild = child->nextSibling();
-      if (!nickInfoList->contains(LocaleString(child->text(0).lower()))) delete \
child; +      if (!nickInfoList->contains(child->text(0).lower())) delete child;
       child = nextChild;
     }
     // List offline nicknames.
@@ -227,7 +227,7 @@ void NicksOnline::updateServerOnlineList(Server* server, bool)
     while (child)
     {
       nextChild = child->nextSibling();
-      if (!nickInfoList->contains(LocaleString(child->text(0).lower()))) delete \
child; +      if (!nickInfoList->contains(child->text(0).lower())) delete child;
       child = nextChild;
     }
     // Expand server if newly added to list.
diff --git a/konversation/server.cpp b/konversation/server.cpp
index 176eaca..46c647e 100644
--- a/konversation/server.cpp
+++ b/konversation/server.cpp
@@ -1068,7 +1068,7 @@ QString Server::getNumericalIp()
 // Given a nickname, returns NickInfo object.   0 if not found.
 NickInfoPtr Server::getNickInfo(const QString& nickname)
 {
-  LocaleString lcNickname(nickname.lower());
+  QString lcNickname(nickname.lower());
   if (allNicks.contains(lcNickname))
   {
     NickInfoPtr nickinfo = allNicks[lcNickname];
@@ -1087,7 +1087,7 @@ NickInfoPtr Server::obtainNickInfo(const QString& nickname)
   if (!nickInfoPtr)
   {
     nickInfoPtr = new NickInfo(nickname, this);
-    allNicks.insert(LocaleString(nickname.lower()), nickInfoPtr);
+    allNicks.insert(QString(nickname.lower()), nickInfoPtr);
   }
   return nickInfoPtr;
 }
@@ -1097,7 +1097,7 @@ NickInfoPtr Server::obtainNickInfo(const QString& nickname)
 // Using code must not alter the list.
 const ChannelNickMap *Server::getJoinedChannelMembers(const QString& channelName) \
const  {
-  LocaleString lcChannelName = channelName.lower();
+  QString lcChannelName = channelName.lower();
   if (joinedChannels.contains(lcChannelName))
     return joinedChannels[lcChannelName];
   else
@@ -1109,7 +1109,7 @@ const ChannelNickMap *Server::getJoinedChannelMembers(const \
QString& channelName  // Using code must not alter the list.
 const ChannelNickMap *Server::getUnjoinedChannelMembers(const QString& channelName) \
const  {
-  LocaleString lcChannelName = channelName.lower();
+  QString lcChannelName = channelName.lower();
   if (unjoinedChannels.contains(lcChannelName))
     return unjoinedChannels[lcChannelName];
   else
@@ -1133,7 +1133,7 @@ const ChannelNickMap *Server::getChannelMembers(const QString& \
channelName) cons  ChannelNickPtr Server::getChannelNick(const QString& channelName, \
const QString& nickname)  {
   QString lcChannelName = channelName.lower();
-  LocaleString lcNickname = nickname.lower();
+  QString lcNickname = nickname.lower();
   const ChannelNickMap *channelNickMap = getChannelMembers(lcChannelName);
   if (channelNickMap)
   {
@@ -1179,7 +1179,7 @@ ChannelNickPtr Server::setChannelNick(const QString& \
channelName, const QString&  // Returns a list of all the channels (joined or \
unjoined) that a nick is in.  QStringList Server::getNickChannels(QString& nickname)
 {
-  LocaleString lcNickname = nickname.lower();
+  QString lcNickname = nickname.lower();
   QStringList channellist;
   ChannelMembershipMap::Iterator channel;
   for( channel = joinedChannels.begin(); channel != joinedChannels.end(); ++channel \
) @@ -1252,7 +1252,7 @@ void Server::addQuery(const QString& nickname,const QString& \
hostmask, bool wein  
 #ifdef USE_NICKINFO
     // Update NickInfo.
-    LocaleString lcNickname = nickname.lower();
+    QString lcNickname = nickname.lower();
     NickInfoPtr nickInfo = obtainNickInfo(nickname);
     if ((nickInfo->getHostmask() != hostmask) && !hostmask.isEmpty())
     {
@@ -1276,7 +1276,7 @@ void Server::closeQuery(const QString &name)
 
 #ifdef USE_NICKINFO
   // Update NickInfo.
-  queryNicks.remove(LocaleString(name.lower()));
+  queryNicks.remove(QString(name.lower()));
 #endif
 }
 
@@ -1838,11 +1838,11 @@ ChannelNickPtr Server::addNickToJoinedChannelsList(const \
QString& channelName, c  bool doChannelJoinedSignal = false;
   bool doWatchedNickChangedSignal = false;
   bool doChannelMembersChangedSignal = false;
-  LocaleString lcNickname = nickname.lower();
+  QString lcNickname = nickname.lower();
   // Create NickInfo if not already created.
   NickInfoPtr nickInfo = obtainNickInfo(nickname);
   // Move the channel from unjoined list (if present) to joined list.
-  LocaleString lcChannelName = channelName.lower();
+  QString lcChannelName = channelName.lower();
   ChannelNickMap *channel;
   if (unjoinedChannels.contains(lcChannelName))
   {
@@ -1923,11 +1923,11 @@ ChannelNickPtr Server::addNickToUnjoinedChannelsList(const \
QString& channelName,  bool doChannelUnjoinedSignal = false;
   bool doWatchedNickChangedSignal = false;
   bool doChannelMembersChangedSignal = false;
-  LocaleString lcNickname = nickname.lower();
+  QString lcNickname = nickname.lower();
   // Create NickInfo if not already created.
   NickInfoPtr nickInfo = obtainNickInfo(nickname);
   // Move the channel from joined list (if present) to unjoined list.
-  LocaleString lcChannelName = channelName.lower();
+  QString lcChannelName = channelName.lower();
   ChannelNickMap *channel;
   if (joinedChannels.contains(lcChannelName))
   {
@@ -1982,7 +1982,7 @@ ChannelNickPtr Server::addNickToUnjoinedChannelsList(const \
QString&, const QStri  NickInfoPtr Server::addNickToOnlineList(const QString& \
nickname)  {
   bool doSignal = false;
-  LocaleString lcNickname = nickname.lower();
+  QString lcNickname = nickname.lower();
   nicknamesOffline.remove(lcNickname);
   NickInfoPtr nickInfo = obtainNickInfo(nickname);
   if (!nicknamesOnline.contains(lcNickname))
@@ -2006,7 +2006,7 @@ NickInfoPtr Server::addNickToOnlineList(const QString&) { \
return 0; }  NickInfoPtr Server::addNickToOfflineList(const QString& nickname, const \
QStringList& watchList)  {
   bool doSignal = false;
-  LocaleString lcNickname = nickname.lower();
+  QString lcNickname = nickname.lower();
   nicknamesOnline.remove(lcNickname);
   NickInfoPtr nickInfo;
   if (watchList.find(lcNickname) != watchList.end())
@@ -2044,8 +2044,8 @@ void Server::removeChannelNick(const QString& channelName, \
const QString& nickna  {
   bool doSignal = false;
   bool joined = false;
-  LocaleString lcChannelName = channelName.lower();
-  LocaleString lcNickname = nickname.lower();
+  QString lcChannelName = channelName.lower();
+  QString lcNickname = nickname.lower();
   ChannelNickMap *channel;
   if (joinedChannels.contains(lcChannelName))
   {
@@ -2095,7 +2095,7 @@ void Server::removeJoinedChannel(const QString& channelName)
   bool doSignal = false;
   QString watchList = KonversationApplication::preferences.getNotifyString();
   QStringList watchListLower = QStringList::split(' ', watchList.lower());
-  LocaleString lcChannelName = channelName.lower();
+  QString lcChannelName = channelName.lower();
   // Move the channel nick list from the joined to unjoined lists.
   ChannelNickMap *channel;
   ChannelNickMap::Iterator member;
@@ -2108,7 +2108,7 @@ void Server::removeJoinedChannel(const QString& channelName)
     bool allDeleted = true;
     for ( member = channel->begin(); member != channel->end() ;++member )
     {
-      LocaleString lcNickname = member.key();
+      QString lcNickname = member.key();
       if (watchListLower.find(lcNickname) == watchListLower.end())
       {
         // Remove the nickname from the unjoined channel.  If nickname is no longer
@@ -2143,9 +2143,9 @@ void Server::renameNickInfo(NickInfoPtr nickInfo, const \
QString& newname)  if (nickInfo)
   {
     // Get existing lowercase nickname and rename nickname in the NickInfo object.
-    LocaleString lcNickname = nickInfo->getNickname().lower();
+    QString lcNickname = nickInfo->getNickname().lower();
     nickInfo->setNickname(newname);
-    LocaleString lcNewname = newname.lower();
+    QString lcNewname = newname.lower();
     // Rename the key in allNicks list.
     allNicks.remove(lcNickname);
     allNicks.insert(lcNewname, nickInfo);
diff --git a/konversation/server.h b/konversation/server.h
index 68b16d2..c678108 100644
--- a/konversation/server.h
+++ b/konversation/server.h
@@ -44,7 +44,6 @@ class KonversationMainWindow;
 class RawLog;
 class ChannelListPanel;
 class ScriptLauncher;
-class LocaleString;
 
 class Server : public QObject
 {


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

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