[EGIT] [core/efl] master 01/01: eolian: remove some more temps

[Daniel Kolesa]

q66 pushed a commit to branch master.

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

commit 212d500016287b1a6a3bc03c66451cd0c7fcef21
Author: Daniel Kolesa <d.kol...@samsung.com>
Date:   Fri Sep 12 13:30:24 2014 +0100

    eolian: remove some more temps
---
 src/lib/eolian/eo_definitions.c |  6 ------
 src/lib/eolian/eo_definitions.h |  2 --
 src/lib/eolian/eo_parser.c      | 23 +++++++----------------
 3 files changed, 7 insertions(+), 24 deletions(-)

diff --git a/src/lib/eolian/eo_definitions.c b/src/lib/eolian/eo_definitions.c
index 4f1b9d6..01643f4 100644
--- a/src/lib/eolian/eo_definitions.c
+++ b/src/lib/eolian/eo_definitions.c
@@ -19,9 +19,6 @@ eo_definitions_temps_free(Eo_Lexer_Temps *tmp)
    EINA_LIST_FREE(tmp->params, par)
      database_parameter_del(par);
 
-   if (tmp->legacy_def)
-     eina_stringshare_del(tmp->legacy_def);
-
    if (tmp->kls)
      database_class_del(tmp->kls);
 
@@ -31,9 +28,6 @@ eo_definitions_temps_free(Eo_Lexer_Temps *tmp)
    EINA_LIST_FREE(tmp->var_defs, var)
      database_var_del(var);
 
-   if (tmp->param)
-     database_parameter_del(tmp->param);
-
    EINA_LIST_FREE(tmp->str_items, s)
      if (s) eina_stringshare_del(s);
 
diff --git a/src/lib/eolian/eo_definitions.h b/src/lib/eolian/eo_definitions.h
index 6fa35ed..8b304c1 100644
--- a/src/lib/eolian/eo_definitions.h
+++ b/src/lib/eolian/eo_definitions.h
@@ -13,11 +13,9 @@ typedef struct _Eo_Lexer_Temps
    Eina_List *classes;
    Eina_List *str_bufs;
    Eina_List *params;
-   Eina_Stringshare *legacy_def;
    Eolian_Class *kls;
    Eina_List *type_defs;
    Eina_List *var_defs;
-   Eolian_Function_Parameter *param;
    Eina_List *str_items;
    Eina_List *expr_defs;
    Eina_List *strs;
diff --git a/src/lib/eolian/eo_parser.c b/src/lib/eolian/eo_parser.c
index b54c4c0..937f295 100644
--- a/src/lib/eolian/eo_parser.c
+++ b/src/lib/eolian/eo_parser.c
@@ -1142,7 +1142,7 @@ parse_param(Eo_Lexer *ls, Eina_Bool allow_inout, 
Eina_Bool is_vals)
    par->base.file = eina_stringshare_ref(ls->filename);
    par->base.line = ls->line_number;
    par->base.column = ls->column;
-   ls->tmp.param = par;
+   ls->tmp.params = eina_list_append(ls->tmp.params, par);
    if (allow_inout)
      {
         if (ls->t.kw == KW_at_in)
@@ -1209,12 +1209,12 @@ parse_param(Eo_Lexer *ls, Eina_Bool allow_inout, 
Eina_Bool is_vals)
 }
 
 static void
-parse_legacy(Eo_Lexer *ls)
+parse_legacy(Eo_Lexer *ls, const char **out)
 {
    eo_lexer_get(ls);
    check_next(ls, ':');
    check(ls, TOK_VALUE);
-   ls->tmp.legacy_def = eina_stringshare_ref(ls->t.value.s);
+   *out = eina_stringshare_ref(ls->t.value.s);
    eo_lexer_get(ls);
    check_next(ls, ';');
 }
@@ -1285,12 +1285,10 @@ parse_accessor(Eo_Lexer *ls, Eolian_Function *prop)
         break;
       case KW_legacy:
         CASE_LOCK(ls, legacy, "legacy name")
-        parse_legacy(ls);
         if (is_get)
-          prop->get_legacy = ls->tmp.legacy_def;
+          parse_legacy(ls, &prop->get_legacy);
         else
-          prop->set_legacy = ls->tmp.legacy_def;
-        ls->tmp.legacy_def = NULL;
+          parse_legacy(ls, &prop->set_legacy);
         break;
       case KW_eo:
         CASE_LOCK(ls, eo, "eo name")
@@ -1313,12 +1311,7 @@ end:
 static void
 parse_params(Eo_Lexer *ls, Eina_Bool allow_inout, Eina_Bool is_vals)
 {
-   PARSE_SECTION
-     {
-        parse_param(ls, allow_inout, is_vals);
-        ls->tmp.params = eina_list_append(ls->tmp.params, ls->tmp.param);
-        ls->tmp.param = NULL;
-     }
+   PARSE_SECTION parse_param(ls, allow_inout, is_vals);
    check_match(ls, '}', '{', line, col);
 }
 
@@ -1490,9 +1483,7 @@ body:
         break;
       case KW_legacy:
         CASE_LOCK(ls, legacy, "legacy name")
-        parse_legacy(ls);
-        meth->get_legacy = ls->tmp.legacy_def;
-        ls->tmp.legacy_def = NULL;
+        parse_legacy(ls, &meth->get_legacy);
         break;
       case KW_eo:
         CASE_LOCK(ls, eo, "eo name")

--