[ 
https://issues.apache.org/jira/browse/CB-8836?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14700147#comment-14700147
 ] 

ASF GitHub Bot commented on CB-8836:
------------------------------------

Github user idpaterson commented on the pull request:

    
https://github.com/apache/cordova-plugin-splashscreen/pull/56#issuecomment-131946308
  
    The `dispatch_async` calls were added 
[here](https://github.com/apache/cordova-plugin-splashscreen/commit/559b300d29ba6eb2f951eb19b9bbf7ba9238a862)
 and reports on the commit of fading no longer working were disregarded. 
Further discussion on [the original 
issue](https://issues.apache.org/jira/browse/CB-8836) noted that the 
`dispatch_async` calls did not fully resolve whatever crash was being 
experienced.
    
    Certainly removing these calls outright is not a complete solution since it 
disregards the earlier problem of crashes due to operating off the main thread, 
but this pull request thus far identifies that changes in the animation block 
must be synchronous in order to actually animate.
    
    The related issue in Jira for fading no longer working on iOS is 
[CB-8875](https://issues.apache.org/jira/browse/CB-8875) for which there is an 
outstanding pull request #44 by @chrskrchr. This is a better but possibly not a 
complete solution either since the calls to `destroyViews` and `createViews` 
should probably also only be called on the main thread. 


> Crashes after animating splashscreen
> ------------------------------------
>
>                 Key: CB-8836
>                 URL: https://issues.apache.org/jira/browse/CB-8836
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin SplashScreen
>         Environment: iOS
>            Reporter: Shazron Abdullah
>            Assignee: Shazron Abdullah
>         Attachments: CRASH LOG 1, CRASH LOG 2
>
>
> Discussion in https://github.com/apache/cordova-plugin-splashscreen/pull/21
> Fix here: 
> https://gist.github.com/shazron/e45705f76bef4448139a/e6e1820943197e48fd6baf44b765ebdea0252b04
> Affected lines:
> https://github.com/apache/cordova-plugin-splashscreen/blob/35272415d3b75440b1ea5fe9a5baab5e1586b3c1/src/ios/CDVSplashScreen.m#L292-L304
> Crash causes:
> 1. retain cycle problem on self in the block
> 2. performing UIKit operations while not in the main thread.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to