Don't call the QString copy constructor without need

This commit is contained in:
Aleix Pol 2015-03-14 02:20:25 +01:00
parent 4c195e1a16
commit 128241c95c

View file

@ -155,9 +155,9 @@ QVariant DevicesModel::data(const QModelIndex& index, int role) const
return QIcon::fromTheme(icon).pixmap(32, 32); return QIcon::fromTheme(icon).pixmap(32, 32);
} }
case IdModelRole: case IdModelRole:
return QString(device->id()); return device->id();
case NameModelRole: case NameModelRole:
return QString(device->name()); return device->name();
case Qt::ToolTipRole: case Qt::ToolTipRole:
return QVariant(); //To implement return QVariant(); //To implement
case StatusModelRole: { case StatusModelRole: {