I ran a test case, it fix a memory leak issue.

update the documentation for .size_formatted, .size_native

@ali.alzyod
I discussed this patch with @woohyun .
The conclusion is that let's update the documentation and give sizes without style paddings as your last suggestion via messenger.
So, I'll update the documentation on this patch.

@ali.alzyod
The description only shows a example when there is no margin and padding.
What about the margin? As I know, the margin value is added to the both sizes(formatted and native)!
I think there is no rule about how to handle style padding in any descriptions.

@ali.alzyod
Actually, the descriptions for formatted size and native size are not clear.
The descriptions only describe about size with or without line wrapping.

The code indentation is WTH!
There are so many TABs in the rest of code.
Because of TAB in other lines, it looks like HECK.

Fix code indentation.

Update summary.

@raster
Thank you for answer.
By the way, the pixel art in your profile picture looks awesome. :)

@raster
I agree that it would break all themes. I'll abandon this anyway. But, still I have doubt and I think it is a bug.

Here is a example:

It looks good. I've seen a quite good performance improvement with this patch.

@ali.alzyod
Thank you for your first patch!
I leaved a inline comment. Please check that comment. :)
And please check the following things.

Add more condition for using xlocale.h

@bu5hm4n
Oh, thank you for testing. :)
Because of firewall of office, I couldn't push my patch to a dev branch directly.
I'll update it soon. Thank you!

@raster, @cedric
Previous patch for eina_convert (D6644) caused a build failure from OS X. :(
So I uploaded this patch. But, I can't run build test on OS X.
Is there a way to test it on OS X using build system?

Use USHRT_MAX to avoid overflow problem itself.

If you want to see a visualized one of the test case, please test it.

fix a comment in test case.

remove an unwanted change from this patch

elementary entry: apply scale to entry_edje when an entry is scrollable

It fixed many issues caused by recent changes in Edje.
I'll accept this patch. Thank you.

add since tag

Remove error/critical messages with/without GL

@cedric
I found you pushed patch to use efl_unref() for removing error messages from efl_del when there is no parent.
But, if you enable GL engine support, fb->buffer will have a parent. So, efl_unref() will print the critical error messages.

Hmm... efl_unref also prints another error messages. I think it is not the perfect solution for the issue.

It needs more legacy alias names. I found the following alias has to be included.

fix build failure

update brief documents for Elm_Textpath.

Uh oh.... I didn't modified copied documents. LOL.
Please, review only its purpose. I'll update documents if the patch is acceptable.
If it is not acceptable, I'll abandon this and maintain this in Tizen platform only.

ping~

@YOhoho
Thank you for patch. There are few things to check.

1. Please , put bracket on both conditions of if in edje_util.c. Normally, EFL puts brackets both side if we needed it.
2. Please, test your new test case with new condition for edje_recalc_do(): avoiding calling edje_recalc_do() when Edje is not dirty.

Init and free locale in eina_init, eina_shutdown

@vtorri
Thank you for good opinion. I'll update it.

So, is this OK?
If I need to check it more, please let me know.

update the document

@vtorri
I don't agree renaming internal functions to new/del. Actually, _get function just return static locale variable. new/free or new/del makes confuse people.

update the following things.

fix build failure from Windows OS

@vtorri
Thank you. I missed your comment in the previous update.

fix build error Windows OS.

add a simple test case

fix build error in Windows OS.

fix a tiny code indentation

remove changes of Evas Textblock

@zmike
Ok. Then, I'll split changes of Evas Textblock into an another patch after applying this.
Also, I'll update commit message with @feature tag.

add eina_convert_strtod_c API and apply it to evas_object_textblock

Thank you for review!

And please check a similar legacy type support from the Efl.Ui.Button.

Thank you for review!
It was generated by Eolian of efl 1.16.0. (Tizen 4.0)
I attached a generated .eo.legacy.h file of the Textblock.

@herdsman
Thank you for quick review! :)
I feel EFL became really hard to maintain recently.
After applying new interfaces into legacy code and legacy API.
It is not your fault but the strategy for new interface implementation.
Anyway, thank you for review!

We are struggling to fix the issues caused by recent upstream patches..
Please, add test cases when you guys try to change existing behaviors even if you are a committer.

I passed the patch file to @smohanty to push it directly.
So, I abandon this.

@herdsman
Thank you for quick review!

Remove an irrelevant change from test_entry.c

@herdsman
Thank you for comment.

Update the following things...

1. Add/Apply a helper function for creating "grapheme_breaks".
2. Add/Apply a helper function for reducing code duplication of prev/next function.