-
Notifications
You must be signed in to change notification settings - Fork 74
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
Don't use enumerateByteRangesUsingBlock #220
Conversation
336ae14
to
1f81135
Compare
This is basically a revert for spotify@844112c It is actually not clear how enumerateByteRangesUsingBlock can fix any crash there but on the other hand is causing unnecessary callback and delay in data delivering.
1f81135
to
c6f47b9
Compare
Codecov Report
@@ Coverage Diff @@
## master #220 +/- ##
==========================================
- Coverage 94.56% 94.55% -0.02%
==========================================
Files 26 26
Lines 1325 1322 -3
==========================================
- Hits 1253 1250 -3
Misses 72 72
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
I'm not familiar with the scenario that the original commit addressed, but it looks like Apple recommends using that approach: https://developer.apple.com/documentation/foundation/nsurlsessiondatadelegate/1411528-urlsession#discussion |
thanks for the link I missed that recommendation. On the other hand if I get it correctly they are talking about when the |
OK, good point — it should be left up to the delegate to enumerate the bytes if needed. |
This is basically a revert for
844112c
It is actually not clear how enumerateByteRangesUsingBlock can fix any
crash there but on the other hand is causing unnecessary callback and
delay in data delivering.