mirrors/metatext
1
0
Fork 0
mirror of https://github.com/metabolist/metatext.git synced 2024-11-25 17:50:59 +00:00
Code Issues Projects Releases Wiki Activity

Ensure no memory leaks caused by image download completion blocks

Browse source
This commit is contained in:
Justin Mazzocchi 2021-04-04 12:34:19 -07:00
parent 47a43e6cb4
commit 36b618c03d
No known key found for this signature in database
GPG key ID: E223E6937AAFB01C
2 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
View Controllers/EditAttachmentViewController.swift
View file

@ -190,13 +190,13 @@ private extension EditAttachmentViewController {
SDWebImageManager.shared.loadImage( SDWebImageManager.shared.loadImage(
with: viewModel.attachment.url.url, with: viewModel.attachment.url.url,
options: [], options: [],
progress: nil) { image, _, _, _, _, _ in progress: nil) { [weak self] image, _, _, _, _, _ in
guard let cgImage = image?.cgImage else { return } guard let self = self, let cgImage = image?.cgImage else { return }
self.detectText(cgImage: cgImage) self.detectText(cgImage: cgImage)
.sink { [weak self] in .sink {
if case let .failure(error) = $0 { if case let .failure(error) = $0 {
self?.present(alertItem: .init(error: error)) self.present(alertItem: .init(error: error))
} }
} receiveValue: { [weak self] in } receiveValue: { [weak self] in
guard let self = self else { return } guard let self = self else { return }

4
View Controllers/ImageViewController.swift
View file

@ -140,11 +140,11 @@ final class ImageViewController: UIViewController {
} }
imageView.sd_setImage(with: viewModel.attachment.url.url, imageView.sd_setImage(with: viewModel.attachment.url.url,
placeholderImage: placeholderImage) { _, error, _, _ in placeholderImage: placeholderImage) { [weak self] _, error, _, _ in
if error != nil { if error != nil {
let alertItem = AlertItem(error: ImageError.unableToLoad) let alertItem = AlertItem(error: ImageError.unableToLoad)
self.present(alertItem: alertItem) self?.present(alertItem: alertItem)
} }
} }
case .gifv: case .gifv: