Apply suggestions from code review

Co-authored-by: Alexey Rusakov <[email protected]>
quotient-im · Apr 17, 2023 · 3d11ddc · 3d11ddc
1 parent 683a974
commit 3d11ddc
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/client/chatroomwidget.cpp b/client/chatroomwidget.cpp
@@ -794,7 +794,7 @@ bool ChatRoomWidget::setReferringMode(const int newMode, const QString& eventId,
                                       const char* icon_name)
 {
     Q_ASSERT( newMode == Replying || newMode == Editing );
-    // Actually, we could let the user to refer to pending events too but in
+    // Actually, we could let the user refer to pending events too but in
     // this case we would need a universal pointer instead of event id. Now the
     // user cannot start to edit a pending message which might be annoying if
     // transactions are acknowledged slowly.

diff --git a/client/models/messageeventmodel.cpp b/client/models/messageeventmodel.cpp
@@ -923,7 +923,7 @@ QVariant MessageEventModel::data(const QModelIndex& idx, int role) const
             evt, [](const RoomCreateEvent& e) { return e.predecessor().roomId; },
             [](const RoomTombstoneEvent& e) { return e.successorRoomId(); });
 
-    if (role == NudeRichBodyRole)
+    if (role == BareRichBodyRole)
     {
         auto e = eventCast<const Quotient::RoomMessageEvent>(&evt);
         if (!e || !e->hasTextContent())
@@ -934,21 +934,21 @@ QVariant MessageEventModel::data(const QModelIndex& idx, int role) const
             QRegularExpression::DotMatchesEverythingOption
         };
         static const QRegularExpression quoteLines("> .*(?:\n|$)");
-        QString nudeBody;
+        QString bareBody;
         if (e->mimeType().name() != "text/plain") {
             // Naïvely assume that it's HTML
             auto htmlBody =
                 static_cast<const MessageEventContent::TextContent*>(e->content())->body;
             auto [cleanHtml, errorPos, errorString] =
                 HtmlFilter::fromMatrixHtml(htmlBody.remove(quoteBlock), m_currentRoom);
             if (errorPos == -1) {
-                nudeBody = cleanHtml;
+                bareBody = cleanHtml;
             }
         }
-        if (nudeBody.isEmpty()) {
-            nudeBody = m_currentRoom->prettyPrint(e->plainBody().remove(quoteLines));
+        if (bareBody.isEmpty()) {
+            bareBody = m_currentRoom->prettyPrint(e->plainBody().remove(quoteLines));
         }
-        return nudeBody;
+        return bareBody;
     }
 
     if (role == QuotationRole)
@@ -974,7 +974,7 @@ QVariant MessageEventModel::data(const QModelIndex& idx, int role) const
     {
         if (isPending)
             return QString();   // Cannot construct event link with unknown eventId
-        QString quotation = data(idx, NudeRichBodyRole).toString();
+        QString quotation = data(idx, BareRichBodyRole).toString();
         if (quotation.isEmpty())
             return QString();
         const auto authorUser = m_currentRoom->user(evt.senderId());

diff --git a/client/models/messageeventmodel.h b/client/models/messageeventmodel.h
@@ -45,7 +45,7 @@ class MessageEventModel: public QAbstractListModel
             RefRole,
             ReactionsRole,
             EventResolvedTypeRole,
-            NudeRichBodyRole,
+            BareRichBodyRole,
             QuotationRole,
             HtmlQuotationRole,
         };