Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Checks if _brick!=nil before calling framesDidLayout(). #176

Merged
merged 1 commit into from
Aug 23, 2017

Conversation

ablokker
Copy link
Contributor

This fixes a run-time crash during dequeing a cell while brick hasn’t yet been set. Checks if it’s an AsynchronouseResizableCell before calling layoutIfNeeded during the attribute pass.

    
This fixes a run-time crash during dequeing a cell while brick hasn’t yet been set.  Checks if it’s an AsynchronouseResizableCell before calling layoutIfNeeded during the attribute pass.
@codecov-io
Copy link

codecov-io commented Aug 14, 2017

Codecov Report

Merging #176 into master will increase coverage by 0.11%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
+ Coverage   93.37%   93.48%   +0.11%     
==========================================
  Files          39       39              
  Lines        4149     4160      +11     
  Branches      339      341       +2     
==========================================
+ Hits         3874     3889      +15     
+ Misses        274      270       -4     
  Partials        1        1
Impacted Files Coverage Δ
Source/Cells/ImageDownloader.swift 79.31% <100%> (-0.69%) ⬇️
Source/Cells/BrickCell.swift 98.58% <100%> (+0.03%) ⬆️
Source/Bricks/Image/ImageBrick.swift 82.23% <100%> (+3.91%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b9a57d...389ef4e. Read the comment docs.

@jay18001 jay18001 merged commit 6370c41 into wayfair-archive:master Aug 23, 2017
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.

3 participants