CameraKit / blurkit-android

The missing Android blurring library. Fast blur-behind layout that parallels iOS.
MIT License
3.72k stars 309 forks source link

Resolve Memory Leaks #37

Closed emersoncloud closed 5 years ago

emersoncloud commented 5 years ago

Fix current memory leaks in the BlurKit library. If you have encountered any, leave a comment below with the details and I'll be sure it gets addressed.

Official Maintainer Ticket: @emersoncloud

emersoncloud commented 5 years ago

From @SergeyBurlaka "Has memory leaks when use in my xml"

<com.wonderkiln.blurkit.BlurLayout 
xmlns:blurkit="http://schemas.android.com/apk/res-auto"-->          
    android:id="@+id/blurLayout"  
    android:layout_width="match_parent"       
    android:layout_height="match_parent"  
    android:layout_gravity="end"
    blurkit:blurRadius="12"
    blurkit:downscaleFactor="0.90"
    blurkit:fps="60" />
emersoncloud commented 5 years ago

From @hugbio:

"Use BlurLayout is not going to cause memory leaks? Because invalidationLoop does not have removeFrameCallback"