ajwillia-ms pushed a commit to branch master.

http://git.enlightenment.org/tools/edi.git/commit/?id=4d466dbc7492a38d61c74d3136060a74533d0d52

commit 4d466dbc7492a38d61c74d3136060a74533d0d52
Author: Andy Williams <a...@andywilliams.me>
Date:   Sun May 31 21:18:18 2015 +0100

    Fix many warnings.
    
    Make the font face const char * and use eina_stringshare
---
 configure.ac                               |  2 +-
 elm_code/src/lib/elm_code_diff_widget.c    |  2 +-
 elm_code/src/lib/elm_code_diff_widget.h    |  3 ++-
 elm_code/src/lib/widget/elm_code_widget.c  | 14 +++++++-------
 elm_code/src/lib/widget/elm_code_widget.eo |  2 +-
 src/bin/edi_config.h                       |  6 +++---
 src/bin/screens/edi_settings_font.c        |  2 +-
 7 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/configure.ac b/configure.ac
index eea4b9a..7740ead 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,4 +1,4 @@
-EFL_VERSION([0], [1], [dev])
+EFL_VERSION([0], [0], [9])
 AC_INIT([edi], [efl_version], [enlightenment-de...@lists.sourceforge.net])
 
 AC_PREREQ([2.65])
diff --git a/elm_code/src/lib/elm_code_diff_widget.c 
b/elm_code/src/lib/elm_code_diff_widget.c
index eba3b37..e07b9cb 100644
--- a/elm_code/src/lib/elm_code_diff_widget.c
+++ b/elm_code/src/lib/elm_code_diff_widget.c
@@ -121,7 +121,7 @@ elm_code_diff_widget_add(Evas_Object *parent, Elm_Code 
*code)
 }
 
 EAPI void
-elm_code_diff_widget_font_set(Evas_Object *widget, char *name, int size)
+elm_code_diff_widget_font_set(Evas_Object *widget, const char *name, int size)
 {
    Elm_Code_Widget *child;
 
diff --git a/elm_code/src/lib/elm_code_diff_widget.h 
b/elm_code/src/lib/elm_code_diff_widget.h
index 895dd55..7a6f2d6 100644
--- a/elm_code/src/lib/elm_code_diff_widget.h
+++ b/elm_code/src/lib/elm_code_diff_widget.h
@@ -23,7 +23,8 @@ extern "C" {
 
 EAPI Evas_Object *elm_code_diff_widget_add(Evas_Object *parent, Elm_Code 
*code);
 
-EAPI void elm_code_diff_widget_font_set(Evas_Object *widget, char *name, int 
size);
+EAPI void elm_code_diff_widget_font_set(Evas_Object *widget, const char *name,
+                                        int size);
 
 /**
  * @}
diff --git a/elm_code/src/lib/widget/elm_code_widget.c 
b/elm_code/src/lib/widget/elm_code_widget.c
index a320c9e..bf63c35 100644
--- a/elm_code/src/lib/widget/elm_code_widget.c
+++ b/elm_code/src/lib/widget/elm_code_widget.c
@@ -1285,23 +1285,23 @@ _elm_code_widget_lines_visible_get(Eo *obj EINA_UNUSED, 
Elm_Code_Widget_Data *pd
 }
 
 EOLIAN static void
-_elm_code_widget_font_set(Eo *obj EINA_UNUSED, Elm_Code_Widget_Data *pd, char 
*name,
-                          Evas_Font_Size size)
+_elm_code_widget_font_set(Eo *obj EINA_UNUSED, Elm_Code_Widget_Data *pd,
+                          const char *name, Evas_Font_Size size)
 {
-   char *face = name;
+   const char *face = name;
    if (!face)
      face = "Mono";
 
    evas_object_textgrid_font_set(pd->grid, face, size * 
elm_config_scale_get());
    if (pd->font_name)
-     free((char *)pd->font_name);
-   pd->font_name = strdup(face);
+     eina_stringshare_del((char *)pd->font_name);
+   pd->font_name = eina_stringshare_add(face);
    pd->font_size = size;
 }
 
 EOLIAN static void
-_elm_code_widget_font_get(Eo *obj EINA_UNUSED, Elm_Code_Widget_Data *pd, char 
**name,
-                          Evas_Font_Size *size)
+_elm_code_widget_font_get(Eo *obj EINA_UNUSED, Elm_Code_Widget_Data *pd,
+                          const char **name, Evas_Font_Size *size)
 {
    if (name)
      *name = strdup((const char *)pd->font_name);
diff --git a/elm_code/src/lib/widget/elm_code_widget.eo 
b/elm_code/src/lib/widget/elm_code_widget.eo
index f430965..61e8b87 100644
--- a/elm_code/src/lib/widget/elm_code_widget.eo
+++ b/elm_code/src/lib/widget/elm_code_widget.eo
@@ -37,7 +37,7 @@ class Elm_Code_Widget (Elm.Layout, Elm_Interface_Atspi_Text)
             @ingroup Style */
          }
          values {
-            name: char *; /*@ The name of the font to load */
+            name: const(char) *; /*@ The name of the font to load */
             size: Evas_Font_Size; /*@ The font size for the widget */
          }
       }
diff --git a/src/bin/edi_config.h b/src/bin/edi_config.h
index 68d868d..e93f1ec 100644
--- a/src/bin/edi_config.h
+++ b/src/bin/edi_config.h
@@ -52,8 +52,8 @@ struct _Edi_Project_Config_Tab
 
 struct _Edi_Project_Config_Launch
 {
-   char *path;
-   char *args;
+   const char *path;
+   const char *args;
 };
 
 struct _Edi_Project_Config
@@ -62,7 +62,7 @@ struct _Edi_Project_Config
 
    struct
      {
-        char *name;
+        const char *name;
         int size;
      } font;
 
diff --git a/src/bin/screens/edi_settings_font.c 
b/src/bin/screens/edi_settings_font.c
index 371b9ab..a7a4331 100644
--- a/src/bin/screens/edi_settings_font.c
+++ b/src/bin/screens/edi_settings_font.c
@@ -152,7 +152,7 @@ _cb_op_font_sel(void *data, Evas_Object *obj EINA_UNUSED, 
void *event EINA_UNUSE
 }
 
 static void
-_cb_op_fontsize_sel(void *data, Evas_Object *obj, void *event EINA_UNUSED)
+_cb_op_fontsize_sel(void *data EINA_UNUSED, Evas_Object *obj, void *event 
EINA_UNUSED)
 {
    int size = elm_slider_value_get(obj) + 0.5;
 

-- 


Reply via email to