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

List:       kde-commits
Subject:    KDE/kdelibs/kdecore
From:       Urs Wolfer <uwolfer () kde ! org>
Date:       2008-12-21 11:45:04
Message-ID: 1229859904.815874.2041.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 899598 by uwolfer:

* typos--
* exclude some qclasses krazy checks
* optimize QString operations

 M  +5 -5      io/kurl.h  
 M  +1 -1      kconfig_compiler/kconfig_compiler.cpp  
 M  +1 -1      kernel/ktoolinvocation_x11.cpp  
 M  +1 -1      util/klibloader.h  


--- trunk/KDE/kdelibs/kdecore/io/kurl.h #899597:899598
@@ -108,7 +108,7 @@
  * context.
  *
  */
-class KDECORE_EXPORT KUrl : public QUrl // krazy:exclude=dpointer (krazy can't deal \
with embedded classes) +class KDECORE_EXPORT KUrl : public QUrl // \
krazy:exclude=dpointer,qclasses (krazy can't deal with embedded classes)  {
 public:
   typedef QMap<QString, QString> MetaDataMap;
@@ -278,7 +278,7 @@
    * Converts from a QUrl.
    * @param u the QUrl
    */
-  KUrl( const QUrl &u );
+  KUrl( const QUrl &u ); //krazy:exclude=qclasses
   /**
    * Constructor allowing relative URLs.
    *
@@ -969,7 +969,7 @@
    * everything is 7 bit (ascii) after being percent-encoded.
    */
   static KDE_DEPRECATED QString encode_string(const QString &str) {
-      return QLatin1String( QUrl::toPercentEncoding( str ) );
+      return QLatin1String( QUrl::toPercentEncoding( str ) ); \
//krazy:exclude=qclasses  }
 
   /**
@@ -984,7 +984,7 @@
    *
    */
   static KDE_DEPRECATED QString encode_string_no_slash(const QString &str) {
-      return QString::fromLatin1( QUrl::toPercentEncoding( str, "/" ) );
+      return QString::fromLatin1( QUrl::toPercentEncoding( str, "/" ) ); \
//krazy:exclude=qclasses  }
 
   /**
@@ -998,7 +998,7 @@
    *
    */
   static KDE_DEPRECATED QString decode_string(const QString &str) {
-      return QUrl::fromPercentEncoding( str.toLatin1() );
+      return QUrl::fromPercentEncoding( str.toLatin1() ); //krazy:exclude=qclasses
   }
 
 
--- trunk/KDE/kdelibs/kdecore/kconfig_compiler/kconfig_compiler.cpp #899597:899598
@@ -686,7 +686,7 @@
   {
     // Adjust name
     paramName = name;
-    name.replace("$("+param+')', QString());
+    name.remove("$("+param+')');
     // Lookup defaults for indexed entries
     for(int i = 0; i <= paramMax; i++)
     {
--- trunk/KDE/kdelibs/kdecore/kernel/ktoolinvocation_x11.cpp #899597:899598
@@ -265,7 +265,7 @@
     // This method should launch a webbrowser, preferrably without doing a mimetype
     // check first, like KRun (i.e. kde-open) would do.
 
-    // In a KDE session, call kfmclient (which honours BrowserApplication) if \
present, +    // In a KDE session, call kfmclient (which honors BrowserApplication) \
                if present,
     // otherwise xdg-open, otherwise kde-open (which does a mimetype check first \
though).  
     // Outside KDE, call xdg-open if present, otherwise fallback to the above logic.
--- trunk/KDE/kdelibs/kdecore/util/klibloader.h #899597:899598
@@ -172,7 +172,7 @@
      * Converts a numerical error code into a human-readable error message
      *
      * @param componentLoadingError the error code, as set using the @p error
-     *                              paramter of createInstance()
+     *                              parameter of createInstance()
      * @return the translated error message describing the error represented
      *         by @p componentLoadingError
      *


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