-Lazy initialization for html escapes lists
-Lower memory consumtion for escapes lists
-Simplify code maintenance by sorting lists on runtime, new items donot need to respect sort order(run time will handle it)
It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/
I am not a big fan of this function. I would prefer to have one level more of indirection that does all the logic of the len, source and compare function then call the function that just does the allocation and sort. All the caller will use the specific function without having to specify an enum.
Why not memcpy?
Same as above. I think split function would be nicer here too. So same coment as for the previous function.
Weird space character.
I think I made mistake by not adding comment to this function
because I am not copying buffer to buffer but instead addresses of struct of one buffer to other buffer.
I use this technique to save memory.
The idea that i have struct contain (escape_string, escape_value,escape_string_len,escape_value_len).
I should add comment to this function, sorry