Merge pull request #192 from AppDevNext/NotifyItemInserted

Use notifyItemInserted instead of notifyDataSetChanged

Author: hannesa2

LogcatCoreLib/src/main/java/info/hannes/logcat/base/LogListAdapter.kt

@@ -14,16 +14,16 @@ import java.util.*
 
 class LogListAdapter(private var completeLogs: MutableList<String>, filter: String) : RecyclerView.Adapter<LogListAdapter.LogViewHolder>() {
 
-    private var currentFilter: Array<out String>? = null
-    var filterLogs: List<String> = ArrayList()
+    private lateinit var currentFilter: Array<out String>
+    var filterLogs = mutableListOf<String>()
 
     init {
         setFilter(filter)
     }
 
     fun setItems(newItems: MutableList<String>) {
         completeLogs = newItems
-        setFilter(*currentFilter!!)
+        setFilter(*currentFilter)
         notifyDataSetChanged()
     }
 
@@ -41,25 +41,20 @@ class LogListAdapter(private var completeLogs: MutableList<String>, filter: Stri
                 }
             }
             include
-        }
+        }.toMutableList()
         notifyDataSetChanged()
     }
 
-    fun addLine(addLine: String) {
-
-        completeLogs.add(completeLogs.size, addLine)
+    fun addLine(line: String) {
+        completeLogs.add(completeLogs.size, line)
 
-        filterLogs = completeLogs.filter { line ->
-            var include = false
-            currentFilter?.let {
-                for (filter in it) {
-                    if (!include && line.contains(filter))
-                        include = true
+        currentFilter.let {
+            for (filter in it)
+                if (line.contains(filter)) {
+                    filterLogs.add(filterLogs.size, line)
+                    notifyItemInserted(filterLogs.size - 1)
                 }
-            }
-            include
         }
-        notifyDataSetChanged()
     }
 
     /**