Merge one static function into another
authorThiago Macieira <thiago.macieira@intel.com>
Fri, 30 Mar 2012 01:19:13 +0000 (22:19 -0300)
committerQt by Nokia <qt-info@nokia.com>
Wed, 4 Apr 2012 15:35:33 +0000 (17:35 +0200)
One static function was only being used by the other, so just merge
them and reduce the work for the compiler.

Change-Id: Ia7a1c46ace6254633450632fae7ab35816ff13bf
Reviewed-by: Stephen Kelly <stephen.kelly@kdab.com>
src/corelib/io/qurl.cpp

index 270a1db..9753474 100644 (file)
@@ -392,24 +392,16 @@ static const ushort * const encodedQueryActions = encodedFragmentActions + 4; //
 
 
 static inline QString
-recode(const QString &input, const ushort *actions, QUrl::ComponentFormattingOptions encoding,
-                    int from, int iend)
+recodeFromUser(const QString &input, const ushort *actions, int from, int to)
 {
     QString output;
     const QChar *begin = input.constData() + from;
-    const QChar *end = input.constData() + iend;
-    if (qt_urlRecode(output, begin, end, encoding, actions))
+    const QChar *end = input.constData() + to;
+    if (qt_urlRecode(output, begin, end,
+                     QUrl::DecodeUnicode | QUrl::DecodeAllDelimiters | QUrl::DecodeSpaces, actions))
         return output;
 
-    return input.mid(from, iend - from);
-}
-
-static inline QString
-recodeFromUser(const QString &input, const ushort *actions, int from, int end)
-{
-    return recode(input, actions,
-                  QUrl::DecodeUnicode | QUrl::DecodeAllDelimiters | QUrl::DecodeSpaces,
-                  from, end);
+    return input.mid(from, to - from);
 }
 
 void QUrlPrivate::appendAuthority(QString &appendTo, QUrl::FormattingOptions options) const