From b75bc27d55b222c9abb354f642ded129fea57dd8 Mon Sep 17 00:00:00 2001 From: Matthew Chen Date: Wed, 22 Aug 2018 16:37:01 -0400 Subject: [PATCH] Respond to CR. --- .../ConversationView/ConversationViewController.m | 4 ++-- Signal/src/ViewControllers/MessageDetailViewController.swift | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Signal/src/ViewControllers/ConversationView/ConversationViewController.m b/Signal/src/ViewControllers/ConversationView/ConversationViewController.m index 79c8d49f0..69676d7d3 100644 --- a/Signal/src/ViewControllers/ConversationView/ConversationViewController.m +++ b/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]; } diff --git a/Signal/src/ViewControllers/MessageDetailViewController.swift b/Signal/src/ViewControllers/MessageDetailViewController.swift index 0662e9f86..1ec9d3b6d 100644 --- a/Signal/src/ViewControllers/MessageDetailViewController.swift +++ b/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