Respond to CR.

Signal/src/ViewControllers/ConversationView/ConversationViewController.m

@@ -4791,11 +4791,11 @@ typedef enum : NSUInteger {
     ]]
                                                  forGroup:self.thread.uniqueId];
 
-    // We need to impose the range restrictions on the mappings immediately to avoid
-    // doing a great deal of unnecessary work and causing a perf hotspot.
     [self.uiDatabaseConnection readWithBlock:^(YapDatabaseReadTransaction *transaction) {
         [self.messageMappings updateWithTransaction:transaction];
     }];
+    // We need to impose the range restrictions on the mappings immediately to avoid
+    // doing a great deal of unnecessary work and causing a perf hotspot.
     [self updateMessageMappingRangeOptions];
 }
 

Signal/src/ViewControllers/MessageDetailViewController.swift

@@ -137,7 +137,7 @@ class MessageDetailViewController: OWSViewController, MediaGalleryDataSourceDele
         view.addSubview(scrollView)
         scrollView.autoPinWidthToSuperview(withMargin: 0)
 
-        if (scrollView.applyInsetsFix()) {
+        if scrollView.applyInsetsFix() {
             scrollView.autoPin(toTopLayoutGuideOf: self, withInset: 0)
         } else {
             scrollView.autoPinEdge(toSuperviewEdge: .top)
@@ -663,7 +663,7 @@ class MessageDetailViewController: OWSViewController, MediaGalleryDataSourceDele
 
         if let audioAttachmentPlayer = self.audioAttachmentPlayer {
             // Is this player associated with this media adapter?
-            if (audioAttachmentPlayer.owner as? ConversationViewItem == viewItem) {
+            if audioAttachmentPlayer.owner as? ConversationViewItem == viewItem {
                 // Tap to pause & unpause.
                 audioAttachmentPlayer.togglePlayState()
                 return