Merge pull request svga#22 from andyliumstar/master

PonyCui · web-flow · commit 0a1e48779bdf · 2018-01-11T16:17:30.000+08:00
SVGAParser unnecessary thread
diff --git a/library/src/main/java/com/opensource/svgaplayer/SVGAParser.kt b/library/src/main/java/com/opensource/svgaplayer/SVGAParser.kt
@@ -102,8 +102,7 @@ class SVGAParser(private val context: Context) {
     fun parse(inputStream: InputStream, cacheKey: String, callback: ParseCompletion) {
         Thread({
             val videoItem = parse(inputStream, cacheKey)
-            Thread({
-                if (videoItem != null) {
+            if (videoItem != null) {
                     Handler(context.mainLooper).post {
                         callback.onComplete(videoItem)
                     }
@@ -113,7 +112,6 @@ class SVGAParser(private val context: Context) {
                         callback.onError()
                     }
                 }
-            }).start()
         }).start()
     }
 
@@ -307,4 +305,4 @@ class SVGAParser(private val context: Context) {
         zipInputStream.close()
     }
 
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -102,8 +102,7 @@ class SVGAParser(private val context: Context) {`
`102`	`102`	`fun parse(inputStream: InputStream, cacheKey: String, callback: ParseCompletion) {`
`103`	`103`	`Thread({`
`104`	`104`	`val videoItem = parse(inputStream, cacheKey)`
`105`		`- Thread({`
`106`		`- if (videoItem != null) {`
	`105`	`+ if (videoItem != null) {`
`107`	`106`	`Handler(context.mainLooper).post {`
`108`	`107`	`callback.onComplete(videoItem)`
`109`	`108`	`}`
`@@ -113,7 +112,6 @@ class SVGAParser(private val context: Context) {`
`113`	`112`	`callback.onError()`
`114`	`113`	`}`
`115`	`114`	`}`
`116`		`- }).start()`
`117`	`115`	`}).start()`
`118`	`116`	`}`
`119`	`117`
`@@ -307,4 +305,4 @@ class SVGAParser(private val context: Context) {`
`307`	`305`	`zipInputStream.close()`
`308`	`306`	`}`
`309`	`307`
`310`		`-}`
	`308`	`+}`