vtorri
/
enventor
forked from enlightenment/enventor
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

search - fix the replace all misbehavior. 

I missed the offset of the delta length between replace and find.
it found the position of the next incorrectly.
now it's fixed.
This commit is contained in:
ChunEon Park 2014-03-05 03:15:33 +09:00
parent f6e9eb495b
commit 70490cec70
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/bin/search.c
View File

@ -41,14 +41,16 @@ replace_all_proc(search_data *sd)
{
const char *find = elm_entry_entry_get(sd->en_find);
if (!find) return;
int find_len = strlen(find);
const char *replace = elm_entry_entry_get(sd->en_replace);
int replace_len = 0;
//Same word. no need to replace
if (replace)
{
if (!strcmp(find, replace) && (find_len == strlen(replace)))
replace_len = strlen(replace);
if (!strcmp(find, replace) && (find_len == replace_len))
return;
}
@ -60,10 +62,11 @@ replace_all_proc(search_data *sd)
char *s = utf8;
int pos;
int delta = replace_len - find_len;
while ((s = strstr(s, find)))
{
pos = s - utf8;
pos = s + (delta * replace_cnt) - utf8;
elm_entry_select_region_set(sd->entry, pos, pos + find_len);
elm_entry_entry_insert(sd->entry, replace);
elm_entry_select_none(sd->entry);