netstar pushed a commit to branch master. http://git.enlightenment.org/apps/ecrire.git/commit/?id=d5848bfda6e726f08ea33145280845ace26535ab
commit d5848bfda6e726f08ea33145280845ace26535ab Author: Alastair Poole <nets...@gmail.com> Date: Thu Apr 8 07:34:11 2021 +0100 ecrire: naming. --- src/bin/Ecrire.h | 9 +++++---- src/bin/main.c | 38 ++++++++++++++++++-------------------- src/bin/ui/alerts.c | 2 +- src/bin/ui/file_related.c | 1 + src/bin/ui/settings.c | 38 +++++++++++++++++++------------------- 5 files changed, 44 insertions(+), 44 deletions(-) diff --git a/src/bin/Ecrire.h b/src/bin/Ecrire.h index b60d2df..e20f461 100644 --- a/src/bin/Ecrire.h +++ b/src/bin/Ecrire.h @@ -18,6 +18,8 @@ typedef struct _Ecrire_Editor Evas_Object *frame; Evas_Object *entry; + Evas_Object *settings_popup; + struct { Elm_Object_Item *copy, *cut, *save, *paste, *undo, *redo; @@ -28,7 +30,6 @@ typedef struct _Ecrire_Editor Elm_Object_Item *copy, *cut, *save, *paste, *undo, *redo; } toolbar; - const char *filename; int unsaved; @@ -45,10 +46,10 @@ typedef struct _Ecrire_Editor Eina_Bool undo_stack_can_merge; } Ecrire_Editor; -void ecrire_editor_open(const char *filename, const char *font_name, int font_size); -void ecrire_editor_font_set_save(Ecrire_Editor *inst, const char *font, int size); +void ecrire_editor_add(const char *filename, const char *font_name, int font_size); +void ecrire_editor_font_save(Ecrire_Editor *inst, const char *font, int size); void ecrire_editor_font_set(Ecrire_Editor *inst, const char *font, int font_size); -void ecrire_editor_save_then(Ecrire_Editor *inst, void *callback_func); +void ecrire_editor_try_save(Ecrire_Editor *inst, void *callback_func); void ecrire_editor_save(Ecrire_Editor *inst, const char *file); #ifdef ENABLE_NLS diff --git a/src/bin/main.c b/src/bin/main.c index 199518a..95fcb63 100644 --- a/src/bin/main.c +++ b/src/bin/main.c @@ -9,13 +9,11 @@ static Eina_Unicode plain_utf8 = EINA_TRUE; -static void print_usage(const char *bin); - /* specific log domain to help debug only ecrire */ int _ecrire_log_dom = -1; static void -_win_del_do(void *data) +_editor_del(void *data) { Ecrire_Editor *inst = data; evas_object_del(inst->win); @@ -391,7 +389,7 @@ _cb_fs_open_done(void *data EINA_UNUSED, Evas_Object *obj EINA_UNUSED, if (!selected) return; - ecrire_editor_open(selected, _ent_cfg->font.name, _ent_cfg->font.size); + ecrire_editor_add(selected, _ent_cfg->font.name, _ent_cfg->font.size); } static void @@ -453,7 +451,7 @@ _cb_open(void *data EINA_UNUSED, Evas_Object *obj EINA_UNUSED, void *event_info } void -ecrire_editor_save_then(Ecrire_Editor *inst, void *callback_func) +ecrire_editor_try_save(Ecrire_Editor *inst, void *callback_func) { if (inst->filename) ecrire_editor_save(inst, inst->filename); @@ -465,7 +463,7 @@ static void _cb_save(void *data, Evas_Object *obj EINA_UNUSED, void *event_info EINA_UNUSED) { Ecrire_Editor *inst = data; - ecrire_editor_save_then(inst, _cb_fs_ecrire_editor_save); + ecrire_editor_try_save(inst, _cb_fs_ecrire_editor_save); } static void @@ -478,14 +476,14 @@ _cb_save_as(void *data, Evas_Object *obj EINA_UNUSED, void *event_info EINA_UNUS static void _cb_new(void *data EINA_UNUSED, Evas_Object *obj EINA_UNUSED, void *event_info EINA_UNUSED) { - ecrire_editor_open(NULL, _ent_cfg->font.name, _ent_cfg->font.size); + ecrire_editor_add(NULL, _ent_cfg->font.name, _ent_cfg->font.size); } static void _cb_close(void *data, Evas_Object *obj EINA_UNUSED, void *event_info EINA_UNUSED) { Ecrire_Editor *inst = data; - _alert_if_need_saving(_win_del_do, inst); + _alert_if_need_saving(_editor_del, inst); } static void @@ -528,7 +526,7 @@ _cb_win_del(void *data, Evas_Object *obj, void *event_info) { Ecrire_Editor *inst = data; (void) data; - _alert_if_need_saving(_win_del_do, inst); + _alert_if_need_saving(_editor_del, inst); } static void @@ -561,9 +559,10 @@ _cb_key_down(void *data, Evas *e EINA_UNUSED, Evas_Object *obj, void *event) void ecrire_editor_font_set(Ecrire_Editor *inst, const char *font, int font_size) { - const Evas_Object *tb = elm_entry_textblock_get(inst->entry); Eina_Strbuf *sbuf; - + const Evas_Object *tb; + + tb = elm_entry_textblock_get(inst->entry); sbuf = eina_strbuf_new(); if (font) @@ -590,7 +589,7 @@ ecrire_editor_font_set(Ecrire_Editor *inst, const char *font, int font_size) } void -ecrire_editor_font_set_save(Ecrire_Editor *inst, const char *font, int size) +ecrire_editor_font_save(Ecrire_Editor *inst, const char *font, int size) { ecrire_editor_font_set(inst, font, size); @@ -629,6 +628,8 @@ _ecrire_toolbar_add(Ecrire_Editor *inst) elm_toolbar_item_separator_set(it, 1); inst->toolbar.undo = elm_toolbar_item_append(tb, "edit-undo", _("Undo"), _cb_undo, inst); inst->toolbar.redo = elm_toolbar_item_append(tb, "edit-redo", _("Redo"), _cb_redo, inst); + it = elm_toolbar_item_append(tb, "separator", "", NULL, NULL); + elm_toolbar_item_separator_set(it, 1); elm_toolbar_item_append(tb, "edit-find-replace", _("Find"), _cb_find, inst); elm_toolbar_item_append(tb, "go-jump", _("Go to..."), _cb_goto_line, inst); it = elm_toolbar_item_append(tb, "separator", "", NULL, NULL); @@ -638,6 +639,7 @@ _ecrire_toolbar_add(Ecrire_Editor *inst) elm_object_item_disabled_set(inst->toolbar.copy, EINA_TRUE); elm_object_item_disabled_set(inst->toolbar.cut, EINA_TRUE); elm_object_item_disabled_set(inst->toolbar.save, EINA_TRUE); + return tb; } @@ -683,7 +685,7 @@ _ecrire_menu_add(Ecrire_Editor *inst) } void -ecrire_editor_open(const char *filename, const char *font_name, int font_size) +ecrire_editor_add(const char *filename, const char *font_name, int font_size) { Evas_Object *win, *pad, *fr, *bx, *entry, *cur_info; Evas_Object *toolbar; @@ -788,7 +790,7 @@ elm_main(int argc, char **argv) switch (c) { case '?': - print_usage(argv[0]); + fprintf(stderr, "Usage: %s [filename]\n", argv[0]); if (isprint (optopt)) { ERR("Unknown option or requires an argument `-%c'.", @@ -834,7 +836,7 @@ elm_main(int argc, char **argv) } } - ecrire_editor_open(filename, _ent_cfg->font.name, _ent_cfg->font.size); + ecrire_editor_add(filename, _ent_cfg->font.name, _ent_cfg->font.size); elm_run(); @@ -843,10 +845,6 @@ elm_main(int argc, char **argv) _ecrire_log_dom = -1; return 0; } + ELM_MAIN() -static void -print_usage(const char *bin) -{ - fprintf(stderr, "Usage: %s [filename]\n", bin); -} diff --git a/src/bin/ui/alerts.c b/src/bin/ui/alerts.c index a68a352..d4d1caa 100644 --- a/src/bin/ui/alerts.c +++ b/src/bin/ui/alerts.c @@ -37,7 +37,7 @@ _save(void *data, Evas_Object *obj EINA_UNUSED, void *event_info EINA_UNUSED) Ecrire_Editor *inst = done_data; evas_object_del(data); - ecrire_editor_save_then(inst, _fs_save_done); + ecrire_editor_try_save(inst, _fs_save_done); } static void diff --git a/src/bin/ui/file_related.c b/src/bin/ui/file_related.c index ed1b790..954e31d 100644 --- a/src/bin/ui/file_related.c +++ b/src/bin/ui/file_related.c @@ -2,6 +2,7 @@ #include <Elementary.h> #include "../Ecrire.h" +#include "ui.h" typedef struct _File_Selector_Data { diff --git a/src/bin/ui/settings.c b/src/bin/ui/settings.c index 0b60ba3..ea4658d 100644 --- a/src/bin/ui/settings.c +++ b/src/bin/ui/settings.c @@ -3,28 +3,25 @@ #include "../Ecrire.h" -Evas_Object *_settings_popup; - typedef struct { - Evas_Object *list; - Evas_Object *fsize; - Evas_Object *dfont_check; + Evas_Object *list; + Evas_Object *fsize; + Evas_Object *dfont_check; Elm_Object_Item *cur_font; } Settings; static Settings _settings; static void -_cb_popup_del(void *data, Evas_Object *obj EINA_UNUSED, - void *event_info EINA_UNUSED) +_cb_popup_hide(void *data, Evas_Object *obj EINA_UNUSED, + void *event_info EINA_UNUSED) { Ecrire_Editor *inst = data; + if (inst->settings_popup) + evas_object_hide(inst->settings_popup); elm_object_focus_set(inst->win, 1); - if (_settings_popup) - evas_object_del(_settings_popup); - _settings_popup = NULL; } static Eina_List * @@ -72,7 +69,7 @@ _cb_font_selected(void *data, Evas_Object *obj EINA_UNUSED, return; eina_stringshare_replace(&inst->font.name, selected); - ecrire_editor_font_set_save(inst, inst->font.name, inst->font.size); + ecrire_editor_font_save(inst, inst->font.name, inst->font.size); } static void @@ -84,7 +81,7 @@ _cb_spinner_delay_changed(void *data, Evas_Object *obj EINA_UNUSED, inst->font.size = elm_spinner_value_get(obj); if (inst->font.name) - ecrire_editor_font_set_save(inst, inst->font.name, inst->font.size); + ecrire_editor_font_save(inst, inst->font.name, inst->font.size); } static void @@ -102,11 +99,11 @@ _cb_check_changed(void *data, Evas_Object *obj, if (enabled) { inst->font.name = NULL; - ecrire_editor_font_set_save(inst, NULL, 0); + ecrire_editor_font_save(inst, NULL, 0); } else if (inst->font.name) { - ecrire_editor_font_set_save(inst, inst->font.name, inst->font.size); + ecrire_editor_font_save(inst, inst->font.name, inst->font.size); } } @@ -115,7 +112,7 @@ _cb_btn_clicked(void *data, Evas_Object *obj, void *event_info EINA_UNUSED) { Ecrire_Editor *inst = data; - _cb_popup_del(inst, NULL, NULL); + _cb_popup_hide(inst, NULL, NULL); } Evas_Object * @@ -123,13 +120,16 @@ ui_settings_open(Evas_Object *parent, Ecrire_Editor *inst) { Evas_Object *pop, *tb, *rec, *bx, *hbx, *btn, *list, *lb, *sp, *ck; - if (_settings_popup) - return _settings_popup; + if (inst->settings_popup) + { + evas_object_show(inst->settings_popup); + return inst->settings_popup; + } - _settings_popup = pop = elm_popup_add(parent); + inst->settings_popup = pop = elm_popup_add(parent); elm_object_style_set(pop, "transparent"); evas_object_size_hint_weight_set(pop, 1, 1); - evas_object_smart_callback_add(pop, "dismissed", _cb_popup_del, inst); + evas_object_smart_callback_add(pop, "dismissed", _cb_popup_hide, inst); elm_object_part_text_set(pop, "title,text", _("Settings")); bx = elm_box_add(pop); --