Optimize simplification of Schema column

mgrojo · mgrojo · commit ff86e525b387 · 2019-12-30T20:48:58.000+01:00
According to documentation https://doc.qt.io/qt-5/qstring.html#simplified > Whitespace means any character for which QChar::isSpace() returns true. > This includes the ASCII characters '\t', '\n', '\v', '\f', '\r', and ' '. So replacing '\n' is redundant and can be removed without effect.
diff --git a/src/DbStructureModel.cpp b/src/DbStructureModel.cpp
@@ -52,7 +52,7 @@ QVariant DbStructureModel::data(const QModelIndex& index, int role) const
         if(index.column() == ColumnName && item->parent() == browsablesRootItem)
             return QString::fromStdString(sqlb::ObjectIdentifier(item->text(ColumnSchema).toStdString(), item->text(ColumnName).toStdString()).toDisplayString());
         else
-            return Settings::getValue("db", "hideschemalinebreaks").toBool() ? item->text(index.column()).replace("\n", " ").simplified() : item->text(index.column());
+            return Settings::getValue("db", "hideschemalinebreaks").toBool() ? item->text(index.column()).simplified() : item->text(index.column());
     case Qt::EditRole:
         return item->text(index.column());
     case Qt::ToolTipRole: {
