diff options
author | Swiftb0y <12380386+Swiftb0y@users.noreply.github.com> | 2021-04-18 20:30:45 +0200 |
---|---|---|
committer | Swiftb0y <12380386+Swiftb0y@users.noreply.github.com> | 2021-04-19 13:08:05 +0200 |
commit | ffaa010a0f82d1c8be47dff427cc311e38c4e9fc (patch) | |
tree | bc86837a14bc5d139dbdb1dab7cec8d6020577e6 /src/preferences | |
parent | 7c5b0de8e4062f8916d9200cc9c21439921604ca (diff) |
ColorPaletteEditorModel: adhere to naming scheme
variables of pointers should be prefixed with `p`
Diffstat (limited to 'src/preferences')
-rw-r--r-- | src/preferences/colorpaletteeditormodel.cpp | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/preferences/colorpaletteeditormodel.cpp b/src/preferences/colorpaletteeditormodel.cpp index ed1046e68e..f0c8d9a096 100644 --- a/src/preferences/colorpaletteeditormodel.cpp +++ b/src/preferences/colorpaletteeditormodel.cpp @@ -19,11 +19,11 @@ QIcon toQIcon(const QColor& color) { return QIcon(pixmap); } -HotcueIndexListItem* toHotcueIndexListItem(QStandardItem* from) { - VERIFY_OR_DEBUG_ASSERT(from->type() == QStandardItem::UserType) { +HotcueIndexListItem* toHotcueIndexListItem(QStandardItem* pFrom) { + VERIFY_OR_DEBUG_ASSERT(pFrom->type() == QStandardItem::UserType) { return nullptr; } - return static_cast<HotcueIndexListItem*>(from); + return static_cast<HotcueIndexListItem*>(pFrom); } } // namespace @@ -71,12 +71,12 @@ bool ColorPaletteEditorModel::setData(const QModelIndex& modelIndex, const QVari if (modelIndex.isValid() && modelIndex.column() == 1) { const bool initialAttemptSuccessful = QStandardItemModel::setData(modelIndex, value, role); - const auto* hotcueIndexListItem = toHotcueIndexListItem(itemFromIndex(modelIndex)); - VERIFY_OR_DEBUG_ASSERT(hotcueIndexListItem) { + const auto* pHotcueIndexListItem = toHotcueIndexListItem(itemFromIndex(modelIndex)); + VERIFY_OR_DEBUG_ASSERT(pHotcueIndexListItem) { return false; } - auto hotcueIndexList = hotcueIndexListItem->getHotcueIndexList(); + auto hotcueIndexList = pHotcueIndexListItem->getHotcueIndexList(); // make sure no index is outside of range DEBUG_ASSERT(std::is_sorted(hotcueIndexList.cbegin(), hotcueIndexList.cend())); @@ -87,13 +87,13 @@ bool ColorPaletteEditorModel::setData(const QModelIndex& modelIndex, const QVari hotcueIndexList.erase(hotcueIndexList.begin(), endLower); for (int i = 0; i < rowCount(); ++i) { - auto* hotcueIndexListItem = toHotcueIndexListItem(item(i, 1)); + auto* pHotcueIndexListItem = toHotcueIndexListItem(item(i, 1)); - if (hotcueIndexListItem) { + if (pHotcueIndexListItem) { if (i == modelIndex.row()) { - hotcueIndexListItem->setHotcueIndexList(hotcueIndexList); + pHotcueIndexListItem->setHotcueIndexList(hotcueIndexList); } else { - hotcueIndexListItem->removeIndicies(hotcueIndexList); + pHotcueIndexListItem->removeIndicies(hotcueIndexList); } } } |