Commit Graph

133 Commits

Author SHA1 Message Date
Garrett Moon
bcb0bd9fc2 Allow maximum progressive render size to be set 2016-01-25 10:53:29 -08:00
Garrett Moon
f8671046d0 Return if radius will be less than FLT_EPSILON 2016-01-25 10:31:07 -08:00
Garrett Moon
1ee0d2563a cleanup 2016-01-25 10:31:07 -08:00
Garrett Moon
38436d021a Remove unneeded notification removal 2016-01-25 10:31:07 -08:00
Garrett Moon
c98d5626f2 Switch to Accelerate for blurring
Core Image crashes were still showing up in our stack traces
even after many speculative fixes. Lets see if accelerate does
better.
2016-01-25 10:31:07 -08:00
Garrett Moon
873e26a183 Allow blurring of progressive to be disabled 2016-01-22 14:14:17 -08:00
Garrett Moon
561251adb9 After continually seeing crashes on createCGImage:fromRect: I'm ready
to try some speculative fixes. This moves the rendering to a serial queue
A bit disappointing, this will be reverted if it doesn't address the crashes.

If it does… well, we'll have more to work on.
2016-01-19 12:38:37 -08:00
Garrett Moon
971049b47a Merge pull request #101 from mRs-/better-swift-support
Better swift support for optionals
2016-01-15 11:40:02 -08:00
Marius Landwehr
3e385dc0c9 switched nonnull to nullable on pin_setDownloadImageOperation for NSUUID and for the deprecated method 2016-01-15 07:33:41 +01:00
Marius Landwehr
bd9006d86e rework on PINURLSessionManager 2016-01-12 21:46:54 +01:00
Marius Landwehr
d283377b0d added better swift support for deprecated methods 2016-01-12 21:46:44 +01:00
Michael Schneider
9b8b0afc6d Pass through setUpdateWithProgress: value to prefixed property
Fixes the setUpdateWithProgress: method that currently calls itself instead of passing the value through to the prefixed pin_updateWithProgress property. This results into a infinite loop and a crash.
2016-01-04 14:56:29 +01:00
Marius Landwehr
a5e36046bc Removed the warnings and create an issue for this. 2015-12-21 21:35:30 +01:00
Marius Landwehr
0cd4a176a3 Did some rework from the feedback on the pull request 2015-12-21 21:35:29 +01:00
Marius Landwehr
b016ecdc25 added swift support for the CategoryManager
I skipped the deprecations because, they are deprecated and we hope no one is using them any more in swift anyway
2015-12-21 21:35:29 +01:00
Marius Landwehr
337027351e added better swift support for UIImage+DecodedImage 2015-12-21 21:35:29 +01:00
Marius Landwehr
ac3f85deae added better swift support to PINDataTaskOperation 2015-12-21 21:35:29 +01:00
Marius Landwehr
377944a1be swift support for PINRemoteImageCallbacks 2015-12-21 21:35:29 +01:00
Marius Landwehr
0502a63081 better swift support for PINRemoteImageDownloadTask 2015-12-21 21:35:29 +01:00
Marius Landwehr
db1e2be8fd PINRemoteImageDownloadTask got the same manager problem 2015-12-21 21:35:29 +01:00
Marius Landwehr
22fe19780c found the weierd bug in PINRemoteImageProcessorTask as well, maybe we should go in deeper… 2015-12-21 21:35:29 +01:00
Marius Landwehr
35abe961d8 PINRemoteImageProcessor is now better with swift 2015-12-21 21:35:29 +01:00
Marius Landwehr
350d3ed0ba setPriority is not implemented in the PINRemoteImageTask, it should be kicked, but that's maybe another pull request then 2015-12-21 21:35:01 +01:00
Marius Landwehr
a638b89e2f cancelWithUUID:manager: is a bit weierd because manager is not needed, maybe it's obsolete? 2015-12-21 21:35:01 +01:00
Marius Landwehr
c99735d5e4 added better swift support for PINRemoteImageTask 2015-12-21 21:35:01 +01:00
Marius Landwehr
69d4a27c7b PINCache is indeed nullable see https://github.com/pinterest/PINRemoteImage/pull/101/files#diff-3c9a124fee465b8e3daf9ca2426d4da4R115 2015-12-21 21:35:00 +01:00
Marius Landwehr
cb29048a63 added more swift nullable, nonnull operators 2015-12-21 21:34:19 +01:00
Marius Landwehr
8921f5e15b Added swift support for PINURLSessionManager 2015-12-21 21:32:38 +01:00
Marius Landwehr
6da303e715 added swift support for PINRemoteImageManagerResult 2015-12-21 21:32:38 +01:00
Marius Landwehr
e8377d2e21 Added swift support for PINProgressiveImage 2015-12-21 21:32:38 +01:00
Marius Landwehr
e71b9ceecf PINCache is indeed nullable see https://github.com/pinterest/PINRemoteImage/pull/101/files#diff-3c9a124fee465b8e3daf9ca2426d4da4R115 2015-12-21 21:32:38 +01:00
Marius Landwehr
ead70e87b4 added more swift nullable, nonnull operators 2015-12-21 21:32:38 +01:00
Marius Landwehr
1c73c140ff added more swift nullable, nonnull operators 2015-12-21 21:31:41 +01:00
Marius Landwehr
93e8edbbe8 cleared a warning for the nonnull extern const of PINRemoteImageManagerErrorDomain 2015-12-21 21:31:41 +01:00
Marius Landwehr
bee20b5887 added better swift support for PINRemoteImageManager.h 2015-12-21 21:31:41 +01:00
Garrett Moon
a19ac9f05b Merge pull request #105 from bsmith11/addSharedImageManagerConfiguration
Support for sharedImageManager with custom configuration
2015-12-15 13:50:34 -08:00
Garrett Moon
b7cace0802 Revert "use NS_EXTENSION_UNAVAILABLE_IOS instead of PIN_APP_EXTENSIONS"
This reverts commit 464bb1921c.
2015-12-14 14:34:27 -08:00
Garrett Moon
08b5b94573 processor property is unused 2015-12-14 14:33:38 -08:00
Garrett Moon
4922e90adf Merge pull request #107 from foursquare/session-auth-challenge
Fix crash in urlSession authentication challenge.
2015-12-09 10:28:13 -08:00
Garrett Moon
2271a04091 Merge pull request #108 from mRs-/NS_EXTENSION_UNAVAILABLE_IOS
use NS_EXTENSION_UNAVAILABLE_IOS instead of PIN_APP_EXTENSIONS
2015-12-09 10:24:54 -08:00
Marius Landwehr
464bb1921c use NS_EXTENSION_UNAVAILABLE_IOS instead of PIN_APP_EXTENSIONS 2015-12-09 07:47:15 +01:00
Brian Dorfman
21e69bcf80 Fix crash in urlSession authentication challenge.
Implement delegate method for when there is a session-level authentication challenge so that the call isn't forwarded to the task version.
Make sure we always strongSelf inside blocks.
2015-12-08 17:32:27 -08:00
Bradley Smith
cc9f900a95 - added sharedImageManager configuration 2015-12-07 16:05:31 -05:00
Garrett Moon
77e3fe563a Need to release image if returning 2015-12-05 14:18:47 -08:00
Garrett Moon
1bac095042 Remove decode for CI progressive images
Previously, I'd speculatively decoded images returned from the progressive CIContext
because I'd read that doing so would move the memory from the GPU to the CPU, hopefully
making drawing faster.

I decided to test this hypothesis and by drawing the un-'decoded' image and the image
directly returned from CIContext. Here are some results:

When decoded is drawn before regular
[22486:30240049] decoded - 'regular': 0.000263
[22486:30240097] decoded - 'regular': 0.000242
[22486:30240306] decoded - 'regular': 0.000092
[22486:30240068] decoded - 'regular': 0.000384
[22486:30240096] decoded - 'regular': 0.000111
[22486:30240074] decoded - 'regular': 0.000089
[22486:30240046] decoded - 'regular': 0.000273
[22486:30240351] decoded - 'regular': 0.000250
[22486:30240307] decoded - 'regular': 0.000313
[22486:30240308] decoded - 'regular': 0.000684
[22486:30240307] decoded - 'regular': 0.000485
[22486:30240069] decoded - 'regular': 0.002414
[22486:30240052] decoded - 'regular': 0.000397
[22486:30240049] decoded - 'regular': 0.000293
[22486:30240351] decoded - 'regular': 0.000145

When regular is drawn before decoded
[22610:30246046] decoded - 'regular': 0.000056
[22610:30246030] decoded - 'regular': 0.000473
[22610:30246029] decoded - 'regular': 0.000138
[22610:30246053] decoded - 'regular': 0.000387
[22610:30246030] decoded - 'regular': -0.000006
[22610:30246026] decoded - 'regular': 0.000714
[22610:30246034] decoded - 'regular': 0.000097
[22610:30246621] decoded - 'regular': 0.000039
[22610:30246024] decoded - 'regular': 0.000121
[22610:30246662] decoded - 'regular': -0.000090
[22610:30246030] decoded - 'regular': -0.000057
[22610:30246619] decoded - 'regular': 0.000443
[22610:30246033] decoded - 'regular': 0.001297
[22610:30246623] decoded - 'regular': -0.000020
[22610:30246057] decoded - 'regular': 0.001249
[22610:30246660] decoded - 'regular': 0.000208
[22610:30246659] decoded - 'regular': 0.000048
[22610:30246709] decoded - 'regular': -0.000327
[22610:30246027] decoded - 'regular': 0.000953
[22610:30246025] decoded - 'regular': 0.001228
[22610:30246659] decoded - 'regular': 0.001141

Decoding the image makes little difference and that the difference shows that
the 'decoded' image actually takes longer to draw.

So, we'll save memory and processing time by returning the CGImage from the
context directly as opposed to attempting to decode it.
2015-12-05 13:09:14 -08:00
Garrett Moon
d09f83379b CIImage doesn't retain its CGImage :( 2015-11-24 11:42:17 -08:00
Garrett Moon
b427105731 Merge pull request #97 from Adlai-Holler/UseNSOptions
Use NS_OPTIONS for PINRemoteImageManagerDownloadOptions
2015-11-23 14:06:56 -08:00
Adlai Holler
e1bfaa0fa1 Revert "Remove explicit options values"
This reverts commit bace9ce8d6.
2015-11-23 13:50:04 -08:00
Adlai Holler
af7bfe3f59 Revert "Fix raw value for .None option"
This reverts commit 897fd4c55f.
2015-11-23 13:49:59 -08:00
Adlai Holler
897fd4c55f Fix raw value for .None option 2015-11-22 10:49:15 -08:00