cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=ec12506a88dffc4758a7a51a5136a356f4501346

commit ec12506a88dffc4758a7a51a5136a356f4501346
Author: Shuhrat Dehkanov <[email protected]>
Date:   Thu Nov 17 16:15:35 2016 -0800

    elm/datetime: default case for default value
    
    Summary:
    This will eliminate unnecessary initialization of local variable in case 
the type is not EFL_UI_CLOCK_TYPE_YEAR.
    Also, ELM_DATETIME_YEAR case is merged with the default case.
    
    Reviewers: singh.amitesh, cedric
    
    Subscribers: minkyu, jpeg
    
    Differential Revision: https://phab.enlightenment.org/D4409
    
    Signed-off-by: Cedric Bail <[email protected]>
---
 src/lib/elementary/elm_datetime.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/lib/elementary/elm_datetime.c 
b/src/lib/elementary/elm_datetime.c
index 4bb670d..ce40c6e 100644
--- a/src/lib/elementary/elm_datetime.c
+++ b/src/lib/elementary/elm_datetime.c
@@ -41,13 +41,10 @@ elm_datetime_format_get(const Evas_Object *obj)
 static Efl_Ui_Clock_Type
 adjust_field_type(Elm_Datetime_Field_Type type)
 {
-   Efl_Ui_Clock_Type ctype = EFL_UI_CLOCK_TYPE_YEAR;
+   Efl_Ui_Clock_Type ctype;
 
    switch(type)
      {
-      case  ELM_DATETIME_YEAR:
-         ctype = EFL_UI_CLOCK_TYPE_YEAR;
-         break;
       case ELM_DATETIME_MONTH:
          ctype = EFL_UI_CLOCK_TYPE_MONTH;
          break;
@@ -63,8 +60,9 @@ adjust_field_type(Elm_Datetime_Field_Type type)
       case ELM_DATETIME_AMPM:
          ctype = EFL_UI_CLOCK_TYPE_AMPM;
          break;
+      case ELM_DATETIME_YEAR:
       default:
-         ;
+         ctype = EFL_UI_CLOCK_TYPE_YEAR;
      }
 
    return ctype;

-- 


Reply via email to