Sharing violation on .journal file about ffimageloading

[molinch]

Weird I never experienced this bug.
Could you give some more hints about your device and how you use the lib maybe?
Also if you manage to upload a repro sample somewhere that would greatly help

[robbederks]

Maybe this is related with our caching issue from my previous issue report. I'll fork the repo and try to add the caching key parameter. If the issue is fixed, I'll let you know.

[molinch]

If you make any progress on that I would be very happy to integrate it 😃

[mgorbach]

I'm experiencing the same problem as well.

[molinch]

Can you please write which Android version & which Xamarin.Android you use?
Also any additional info like a reproducible sample will help to debug this.

[mgorbach]

Here's my code:

                if (imageLoadingWork != null)
                {
                    imageLoadingWork.Cancel();
                    imageLoadingWork = null;
                }

later ...

                        if (imageUrl != null)
                        {
                            if (shownImageUrl == imageUrl)
                            {
                                // Nothing to do here, we are already showing the correct image
                                ContentImageView.Alpha = 1.0f;
                            }
                            else
                            {
                                // Clear the image and start loading the next one
                                ContentImageView.Image = null;
                                ContentImageView.Alpha = 0.0f;

                                var urlString = imageUrl.ToString();
                                imageLoadingWork = ImageService.LoadUrl(urlString).Finish(work =>
                                    {
                                        if (work.Completed)
                                        {
                                            shownImageUrl = imageUrl;
                                            UIView.AnimateNotify(
                                                AnimationDuration, 
                                                0.0f, 
                                                UIViewAnimationOptions.CurveEaseInOut,
                                                () =>
                                                {
                                                    ContentImageView.Alpha = 1.0f;
                                                }, 
                                                null);
                                        }
                                        else if (work.IsCancelled)
                                        {
                                            // Nothing to do here.
                                        }
                                        else
                                        {
                                            // There seems to be a third case here that indicates successful image loading. Handle it as such.
                                            shownImageUrl = imageUrl;
                                            ContentImageView.Alpha = 1.0f;
                                        }
                                    }).Error(e => Console.WriteLine("Failed to load image: " + urlString + " error: " + e))
                                    .Into(ContentImageView);
                            }
                        }

[molinch]

Thanks for the code. If you are able to upload somewhere, or email, the whole sample that would greatly help.

Which Android version & Xamarin.Android please?

[mgorbach]

This is Xamarin.iOS, iOS 8.4, FFImageLoading 1.1.3, Xamarin.iOS 8.10.1 Business Edition

[mgorbach]

I don't have an isolated sample I can share right now but I'll try to work one up when I have some free time.

[molinch]

Can you send the stacktrace too please?

[molinch]

Could you tell me if df06d7f fixes your issue?

[molinch]

If you give us some feedback on this so we can deliver a new Nuget package.
Thanks!

[molinch]

Fixed and a new Nuget package has been pushed.