From c2595279be57e143ff3a18199f7c867e3a34ab38 Mon Sep 17 00:00:00 2001 From: SangHyeon Jade Lee Date: Wed, 22 Nov 2017 14:12:00 +0900 Subject: [PATCH] genlist: fix item calculation performance issue in calc(), we only need to process first viewport items, but currently it forcely calculate block sized items in every single calc() funciton calls because of || operator. it must be changed as && operator for orignal purpose. Change-Id: If57081046d67c09d66c360dbbb75b31101ecc1e6 Signed-off-by: SangHyeon Jade Lee --- src/mobile_lib/elm_genlist.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mobile_lib/elm_genlist.c b/src/mobile_lib/elm_genlist.c index 69a7305..b4d4912 100644 --- a/src/mobile_lib/elm_genlist.c +++ b/src/mobile_lib/elm_genlist.c @@ -1432,7 +1432,7 @@ _calc(void *data) // TIZEN_ONLY(20150828) : Calculate item which position on viewport area while (sd->queue && - ((sd->viewport_h > processed_size) || (cnt < MAX_ITEMS_PER_BLOCK))) + ((sd->viewport_h > processed_size) && (cnt < MAX_ITEMS_PER_BLOCK))) { Elm_Gen_Item *tmp; tmp = eina_list_data_get(sd->queue); -- 2.7.4