Skip to content

Retain cycling issues fixed#19

Open
Adiletkozha wants to merge 3 commits intoApplikeySolutions:masterfrom
Adiletkozha:Ad_fix_branch
Open

Retain cycling issues fixed#19
Adiletkozha wants to merge 3 commits intoApplikeySolutions:masterfrom
Adiletkozha:Ad_fix_branch

Conversation

@Adiletkozha
Copy link

  1. There was retain cycle in line inside PlayerSlider.swift:
    issue:var delegate: PlayerSliderProtocol?
    fix: weak delegate: PlayerSliderProtocol?
  2. Problem also related to retain cycles inside block which is inside prefetchItems function within PandoraPlayer.swift
    issue: item = DispatchWorkItem(block: {
    fix: item = DispatchWorkItem(block: {[unowned self] in
  3. targetColumn value inside CenterCellCollectionViewFlowLayout had been computing incorrectly
    issue: let targetColumn = floor(proposedContentOffset.x/w + 1)
    fix: let targetColumn = floor(proposedContentOffset.x/(w+self.minimumLineSpacing) + 1)

@Adiletkozha Adiletkozha changed the title Ad fix branch Retain cycling issues fixed Apr 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant