Skip to content
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

thumbnail regeneration resolved #190

Closed
wants to merge 1 commit into from

Conversation

iswiftshek
Copy link
Contributor

Previously, new thumbnails for files were being generated every time we opened a folder and thus it would take time for thumbnails (audio, video, documents) to load.

Now, thumbnails are generated only once and then loaded from cache whenever called. It can be verified in the logs.

@cpg
Copy link
Member

cpg commented Jun 29, 2019

I don't know for sure, but I think this may not be working well. I get lots of these after re-visiting a page and there is

AmahiAnywhere(19456,0x170163000) malloc: *** set a breakpoint in malloc_error_break to debug
AmahiAnywhere(19456,0x1701ef000) malloc: can't allocate region
*** mach_vm_map(size=2572386304) failed (error code=3)
AmahiAnywhere(19456,0x1701ef000) malloc: *** set a breakpoint in malloc_error_break to debug
AmahiAnywhere(19456,0x1701ef000) malloc: can't allocate region
*** mach_vm_map(size=1763590144) failed (error code=3)
AmahiAnywhere(19456,0x1701ef000) malloc: *** set a breakpoint in malloc_error_break to debug
AmahiAnywhere(19456,0x1701ef000) malloc: can't allocate region
*** mach_vm_map(size=1983725568) failed (error code=3)
AmahiAnywhere(19456,0x1701ef000) malloc: *** set a breakpoint in malloc_error_break to debug
AmahiAnywhere(19456,0x1701ef000) malloc: can't allocate region
*** mach_vm_map(size=1763590144) failed (error code=3)
AmahiAnywhere(19456,0x1701ef000) malloc: *** set a breakpoint in malloc_error_break to debug

in fact, I have been able to get to a folder, see all the thumbnails for the music, then browse down and up and some music does no longer have thumbnails. I think I see the malloc messages above in sync with these issues starting, but I am not sure.

@iswiftshek
Copy link
Contributor Author

The disappearance of thumbnails on scrolling.. is it happening specifically on this pr?
Also, does it happen for the video files or any other type of file?

@cpg
Copy link
Member

cpg commented Jul 2, 2019

It happens for mp3 files. I did not try with other types.

It does not seem to happen in the beta branch. Check with a folder full of mp3s.

@cpg
Copy link
Member

cpg commented Jul 10, 2019

Where are we on this one?

@iswiftshek
Copy link
Contributor Author

Does the app also crash/slow down (become less responsive) with those malloc errors?

@cpg
Copy link
Member

cpg commented Jul 26, 2019

Where are we on this?

@iswiftshek
Copy link
Contributor Author

This PR has conflicts, so I am closing it
I will look into it and send another one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants