Don't quote as part of SQL range-based functions

This was leading to double quoting for some types. Now, we leave it to
the caller to ensure that the values passed have the correct quoting
alread applied.

Author: oblivioncth

lib/sql/include/qx/sql/__private/qx-sqlstring_helpers.h

@@ -56,14 +56,18 @@ void appendKeywordParen(QString& str, const QString& word, const R& range)
     /* The boxing here is inefficient, but I'm not sure how to improve the situation since
      * we rely on the SqlString ctor to reliably get a string from value_type.
      */
-    QString rStr = u"'"_s;
+
+    /* NOTE: With current implementation it is up to the caller to ensure that the elements
+     * in 'range' end up quoted as required (or of a type that automatically quotes them
+     * during construction).
+     */
+    QString rStr;
     for(auto n = std::size(range); const auto& value : range)
     {
         rStr += __private::qstring_compat(value);
         if(n-- != 1)
-            rStr += u"','"_s;
+            rStr += u","_s;
     }
-    rStr += u"'"_s;
 
     appendKeywordParen(str, word, rStr);
 }

lib/sql/src/qx-sqlquery.cpp

@@ -131,7 +131,7 @@
 /*!
  *  @def QX_SQL_MEMBER_OVERRIDE()
  *
- *  Used to define a member/field specific value parsing/serialzing override for a
+ *  Used to define a member/field specific value parsing/serializing override for a
  *  SQL-tried struct. The specified member will be processed using the provided
  *  functions instead of potentially available generic ones for that type.
  *
@@ -289,7 +289,7 @@ void SqlQuery::bindValue(const QString& placeholder, const QVariant& val) { mBin
 /*!
  *  @fn Derived& AbstractSqlQuery::IN(const R& range)
  *
- *  Adds an `IN` clause to the query using all of the elments from @a range.
+ *  Adds an `IN` clause to the query using all of the elements from @a range.
  *
  *  If @a range is empty, the SQL statement will be ill-formed.
  */