> Renato,
> 
> Can you please test that proposal of that NetworkListWorker creating its own
> instance of the parent item?
> 
> There is a proposal https://code.launchpad.net/~carlos-mazieri/ubuntu-
> filemanager-app//fix-network-crash-1609051
> 
> Also renamed "parent" to "parentItemInfo".

Thanks, Carlos will be a pleasure test that. 

But as mentioned before if the parent get destroyed the operation will not be 
canceled and will be executed until the end. (In my opinion this is a wast of 
resources, but if you are ok with that)

Based on that information I do not think you need to check for 
(m_parentItemInfo != 0) in line 61. Since m_parentItemInfo will never be null.


-- 
https://code.launchpad.net/~renatofilho/ubuntu-filemanager-app/fix-network-crash/+merge/314870
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to     : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to