diff options
| author | Hosang Kim <hosang12.kim@samsung.com> | 2015-08-25 20:05:57 +0900 |
|---|---|---|
| committer | Amitesh Singh <amitesh.sh@samsung.com> | 2015-08-25 21:11:20 +0530 |
| commit | 68aa6a5cb7f9b720d4a2342cee729f1328ede844 (patch) | |
| tree | 297d878a3c55a67d5d7554964fca435fcd435ba5 | |
| parent | 40e0ea58e1d268383b47459cdd3279203f0b60f0 (diff) | |
| download | elementary-68aa6a5cb7f9b720d4a2342cee729f1328ede844.tar.gz | |
progressbar: fix memory leak
Summary: when elm_progressbar_value_set is called, progress_status is always created.
Test Plan: elementary_test -> progressbar
Reviewers: woohyun, Hermet, cedric, raster
Differential Revision: https://phab.enlightenment.org/D2986
| -rw-r--r-- | src/lib/elm_progressbar.c | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/src/lib/elm_progressbar.c b/src/lib/elm_progressbar.c index 1fbaab36c..aa34ec02f 100644 --- a/src/lib/elm_progressbar.c +++ b/src/lib/elm_progressbar.c @@ -424,20 +424,9 @@ _elm_progressbar_part_value_get(const Eo *obj EINA_UNUSED, Elm_Progressbar_Data EOLIAN static void _elm_progressbar_value_set(Eo *obj, Elm_Progressbar_Data *sd, double val) { - Elm_Progress_Status *ps; - if (sd->val == val) return; - sd->val = val; - if (sd->val < MIN_RATIO_LVL) sd->val = MIN_RATIO_LVL; - if (sd->val > MAX_RATIO_LVL) sd->val = MAX_RATIO_LVL; - - ps = _progress_status_new("elm.cur.progressbar", sd->val); - sd->progress_status = eina_list_append(sd->progress_status, ps); - _val_set(obj); - _units_set(obj); - eo_do(obj, eo_event_callback_call - (ELM_PROGRESSBAR_EVENT_CHANGED, NULL)); + elm_progressbar_part_value_set(obj, "elm.cur.progressbar", val); } EOLIAN static double |