Page MenuHomePhabricator

id213sin (Youngbok Shin)
Text, Font, Edje, Elementary

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Apr 15 2013, 2:09 AM (397 w, 4 d)
Availability
Available

Recent Activity

Wed, Nov 11

id213sin accepted D12187: evas_textblock: reduce content fit calculations.

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

Wed, Nov 11, 2:42 AM · efl

Oct 20 2020

id213sin added a reviewer for D12183: evas/textblock: apply style paddings in fit calculation: ali.alzyod.
Oct 20 2020, 10:05 PM · efl
id213sin requested review of D12183: evas/textblock: apply style paddings in fit calculation.
Oct 20 2020, 10:04 PM · efl

Sep 24 2020

id213sin requested review of D12163: evas: sw font draw - protect against null pointer access.
Sep 24 2020, 2:12 AM · efl

Sep 4 2020

id213sin requested review of D12131: evas/textblock: fix a memory issue caused by a freed ellipsis item.
Sep 4 2020, 1:28 AM · efl

Aug 28 2020

id213sin updated the diff for D12110: evas_textblock: remove style padding from native width and formatted height.

update the documentation for .size_formatted, .size_native

Aug 28 2020, 2:24 AM · efl
id213sin added a comment to D12110: evas_textblock: remove style padding from native width and formatted height.

@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.

Aug 28 2020, 1:15 AM · efl

Aug 25 2020

id213sin added a comment to D12110: evas_textblock: remove style padding from native width and formatted height.

@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.

Aug 25 2020, 12:56 AM · efl

Aug 24 2020

id213sin added a comment to D12110: evas_textblock: remove style padding from native width and formatted height.

@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.

Aug 24 2020, 1:39 AM · efl

Aug 20 2020

id213sin requested review of D12110: evas_textblock: remove style padding from native width and formatted height.
Aug 20 2020, 2:21 AM · efl

Aug 3 2020

id213sin added a comment to D12081: embryo: fix a integer(cell) overflow problem.

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.

Aug 3 2020, 4:14 AM · efl
id213sin updated the diff for D12081: embryo: fix a integer(cell) overflow problem.

Fix code indentation.

Aug 3 2020, 4:12 AM · efl
id213sin added reviewers for D12081: embryo: fix a integer(cell) overflow problem: cedric, woohyun, raster.
Aug 3 2020, 3:49 AM · efl
id213sin updated the diff for D12081: embryo: fix a integer(cell) overflow problem.

Update summary.

Aug 3 2020, 3:47 AM · efl
id213sin requested review of D12081: embryo: fix a integer(cell) overflow problem.
Aug 3 2020, 3:36 AM · efl

Jul 16 2020

id213sin added a comment to D12053: evas: fix over repeated shadow color for soft shadow effect.

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

Jul 16 2020, 2:56 AM · efl

Jul 15 2020

id213sin abandoned D12053: evas: fix over repeated shadow color for soft shadow effect.

@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.

Jul 15 2020, 10:09 PM · efl
id213sin added a comment to D12053: evas: fix over repeated shadow color for soft shadow effect.

Here is a example:

Jul 15 2020, 12:11 AM · efl
id213sin edited reviewers for D12053: evas: fix over repeated shadow color for soft shadow effect, added: ali.alzyod; removed: ali.
Jul 15 2020, 12:08 AM · efl
id213sin requested review of D12053: evas: fix over repeated shadow color for soft shadow effect.
Jul 15 2020, 12:00 AM · efl

Jun 16 2020

id213sin accepted D11984: elementary/perf: reduce number of efl_isa() call. in elementary test application efl_isa() shows up as hotsport during profiling so refactored the code to reduce redundant efl_isa() calls..

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

Jun 16 2020, 10:49 PM · efl

Dec 16 2018

id213sin added a comment to D7461: optimize glyph images data copy into 4 byte aligned images.

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

Dec 16 2018, 11:58 PM · efl

Dec 6 2018

id213sin added a reviewer for D7427: evas: remove memory leaks from deleted Textblock objects: herdsman.
Dec 6 2018, 10:51 PM · efl
id213sin requested review of D7427: evas: remove memory leaks from deleted Textblock objects.
Dec 6 2018, 10:27 PM · efl

Dec 5 2018

id213sin closed D7395: eina: fix a build failure caused by missing 'locale_t' from OSX.
Dec 5 2018, 2:18 AM · efl
id213sin retitled D7418: elm textpath: reduces differences between actual pos and modified pos from elementary textpath: reduces differences between actual position to elm textpath: reduces differences between actual pos and modified pos.
Dec 5 2018, 1:21 AM · efl

Dec 4 2018

id213sin updated the diff for D7395: eina: fix a build failure caused by missing 'locale_t' from OSX.

Add more condition for using xlocale.h

Dec 4 2018, 11:56 PM · efl
id213sin requested review of D7418: elm textpath: reduces differences between actual pos and modified pos.
Dec 4 2018, 10:59 PM · efl
id213sin added a comment to D7395: eina: fix a build failure caused by missing 'locale_t' from OSX.

@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!

Dec 4 2018, 10:36 PM · efl

Nov 30 2018

id213sin added a comment to D7395: eina: fix a build failure caused by missing 'locale_t' from OSX.

@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?

Nov 30 2018, 2:18 AM · efl
id213sin added reviewers for D7395: eina: fix a build failure caused by missing 'locale_t' from OSX: raster, cedric, Hermet.
Nov 30 2018, 2:17 AM · efl
id213sin requested review of D7395: eina: fix a build failure caused by missing 'locale_t' from OSX.
Nov 30 2018, 2:11 AM · efl

Nov 13 2018

id213sin updated the diff for D7264: edje: fix an overflow issue for state values.

Use USHRT_MAX to avoid overflow problem itself.

Nov 13 2018, 12:26 AM · efl
id213sin added a comment to D7264: edje: fix an overflow issue for state values.


This is an example.
If you want to check the issue caused by overflow, please run this patch.
Before test it, you need to enable EDJE_CALC_CACHE.

Nov 13 2018, 12:17 AM · efl
id213sin added a reviewer for D7264: edje: fix an overflow issue for state values: Hermet.
Nov 13 2018, 12:14 AM · efl
id213sin requested review of D7264: edje: fix an overflow issue for state values.
Nov 13 2018, 12:12 AM · efl

Oct 24 2018

id213sin added reviewers for D7204: evas textblock: remove white space after line-break by a next item: subodh, subodh6129.
Oct 24 2018, 4:19 AM · efl
id213sin added a comment to D7204: evas textblock: remove white space after line-break by a next item.

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

Oct 24 2018, 12:20 AM · efl
id213sin updated the diff for D7204: evas textblock: remove white space after line-break by a next item.

fix a comment in test case.

Oct 24 2018, 12:18 AM · efl
id213sin requested review of D7204: evas textblock: remove white space after line-break by a next item.
Oct 24 2018, 12:17 AM · efl

Oct 3 2018

id213sin requested review of D7140: elementary entry: apply scale to all edje objects.
Oct 3 2018, 11:17 PM · efl
id213sin updated the diff for D7137: eina debug: fix a double unlock issue.

remove an unwanted change from this patch

Oct 3 2018, 11:14 PM · efl
id213sin updated the diff for D7137: eina debug: fix a double unlock issue.

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

Oct 3 2018, 11:12 PM · efl
id213sin requested review of D7137: eina debug: fix a double unlock issue.
Oct 3 2018, 9:27 PM · efl

Sep 28 2018

id213sin accepted D6961: edje: call recalc in part_text_get if there is no available text data.

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

Sep 28 2018, 12:04 AM · efl

Sep 27 2018

id213sin updated the diff for D7033: elementary textpath: support legacy APIs.

add since tag

Sep 27 2018, 9:05 PM · efl
id213sin added reviewers for D7119: evas filter: remove critical messages from Evas Filter: Hermet, raster, woohyun.
Sep 27 2018, 8:48 PM · efl
id213sin updated the diff for D7119: evas filter: remove critical messages from Evas Filter.

Remove error/critical messages with/without GL

Sep 27 2018, 8:47 PM · efl
id213sin added a comment to D7119: evas filter: remove critical messages from Evas Filter.

@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.

Sep 27 2018, 8:46 PM · efl
id213sin added a reviewer for D7119: evas filter: remove critical messages from Evas Filter: cedric.
Sep 27 2018, 8:39 PM · efl
id213sin added a comment to D7119: evas filter: remove critical messages from Evas Filter.

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

Sep 27 2018, 8:23 PM · efl
id213sin requested review of D7119: evas filter: remove critical messages from Evas Filter.
Sep 27 2018, 8:17 PM · efl
id213sin added a comment to D7085: eolian: Set widget and widget item in the same group in doxygen.

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

Sep 27 2018, 2:00 AM · efl

Sep 21 2018

id213sin requested review of D7095: elementary_test: use snprintf to prevent memory overflow in test_efl_ui_text.
Sep 21 2018, 3:26 AM · efl
id213sin closed D7086: elementary: remove meaningless memory allocation and leaking.
Sep 21 2018, 12:31 AM · efl
id213sin requested review of D7086: elementary: remove meaningless memory allocation and leaking.
Sep 21 2018, 12:23 AM · efl
id213sin requested review of D7084: evas cache: add more null check for cache.
Sep 21 2018, 12:15 AM · efl
id213sin requested review of D7083: elementary: fix memory leak from Efl.Ui.Layout.Object.
Sep 21 2018, 12:02 AM · efl

Sep 20 2018

id213sin updated the diff for D7033: elementary textpath: support legacy APIs.

fix build failure

Sep 20 2018, 10:47 PM · efl

Sep 17 2018

id213sin updated the diff for D7033: elementary textpath: support legacy APIs.

update brief documents for Elm_Textpath.

Sep 17 2018, 4:43 AM · efl
id213sin added a comment to D7033: elementary textpath: support legacy APIs.

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.

Sep 17 2018, 12:37 AM · efl
id213sin requested review of D7033: elementary textpath: support legacy APIs.
Sep 17 2018, 12:33 AM · efl

Sep 13 2018

id213sin added a reviewer for D7031: edje: add json file support feature: smohanty.
Sep 13 2018, 11:15 PM · efl
id213sin requested review of D7031: edje: add json file support feature.
Sep 13 2018, 10:41 PM · efl

Sep 10 2018

id213sin added a comment to D6644: eina: add locale-independent eina_convert_strtod_c function.

ping~

Sep 10 2018, 6:28 PM · efl

Sep 9 2018

id213sin added a comment to D6961: edje: call recalc in part_text_get if there is no available text data.

@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.
Sep 9 2018, 9:28 PM · efl

Aug 21 2018

id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

Init and free locale in eina_init, eina_shutdown

Aug 21 2018, 6:53 PM · efl
id213sin added a comment to D6644: eina: add locale-independent eina_convert_strtod_c function.

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

Aug 21 2018, 6:28 PM · efl

Aug 13 2018

id213sin added a comment to D6644: eina: add locale-independent eina_convert_strtod_c function.

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

Aug 13 2018, 6:53 PM · efl

Jul 30 2018

id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

update the document

Jul 30 2018, 3:37 AM · efl
id213sin added a comment to D6644: eina: add locale-independent eina_convert_strtod_c function.

@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.

Jul 30 2018, 2:31 AM · efl
id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

update the following things.

Jul 30 2018, 2:00 AM · efl

Jul 29 2018

id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

fix build failure from Windows OS

Jul 29 2018, 9:06 PM · efl

Jul 26 2018

id213sin added a comment to D6644: eina: add locale-independent eina_convert_strtod_c function.

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

Jul 26 2018, 5:43 AM · efl
id213sin updated the test plan for D6644: eina: add locale-independent eina_convert_strtod_c function.
Jul 26 2018, 5:43 AM · efl
id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

fix build error Windows OS.

Jul 26 2018, 5:42 AM · efl
id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

add a simple test case

Jul 26 2018, 5:33 AM · efl
id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

fix build error in Windows OS.

Jul 26 2018, 3:53 AM · efl

Jul 25 2018

id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

fix a tiny code indentation

Jul 25 2018, 9:07 PM · efl
id213sin updated the summary of D6644: eina: add locale-independent eina_convert_strtod_c function.
Jul 25 2018, 9:05 PM · efl
id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

remove changes of Evas Textblock

Jul 25 2018, 9:00 PM · efl
id213sin added a comment to D6644: eina: add locale-independent eina_convert_strtod_c function.

@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.

Jul 25 2018, 8:09 PM · efl

Jul 24 2018

id213sin updated the summary of D6644: eina: add locale-independent eina_convert_strtod_c function.
Jul 24 2018, 10:20 PM · efl
id213sin updated the diff for D6644: eina: add locale-independent eina_convert_strtod_c function.

add eina_convert_strtod_c API and apply it to evas_object_textblock

Jul 24 2018, 10:19 PM · efl

Jul 23 2018

id213sin added a comment to D6644: eina: add locale-independent eina_convert_strtod_c function.

Thank you for review!

Jul 23 2018, 6:44 PM · efl

Jul 20 2018

id213sin added a reviewer for D6644: eina: add locale-independent eina_convert_strtod_c function: subodh6129.
Jul 20 2018, 4:14 AM · efl
id213sin updated the summary of D6644: eina: add locale-independent eina_convert_strtod_c function.
Jul 20 2018, 4:14 AM · efl

Jul 11 2018

id213sin added a comment to D6529: evas textblock: adds missing legacy types.

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

Jul 11 2018, 9:03 PM · efl
id213sin added a comment to D6529: evas textblock: adds missing legacy types.

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.

Jul 11 2018, 8:56 PM · efl
id213sin updated the summary of D6529: evas textblock: adds missing legacy types.
Jul 11 2018, 8:51 PM · efl

Jul 8 2018

Herald added a reviewer for D6529: evas textblock: adds missing legacy types: devilhorns.
Jul 8 2018, 5:51 PM · efl

Jun 22 2018

id213sin added a comment to D6364: edje: fix text set/get issue without edje calculation.

@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!

Jun 22 2018, 2:40 AM · efl
id213sin added a comment to D6364: edje: fix text set/get issue without edje calculation.

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.

Jun 22 2018, 2:03 AM · efl
Herald added a reviewer for D6364: edje: fix text set/get issue without edje calculation: devilhorns.
Jun 22 2018, 1:58 AM · efl

Jun 15 2018

id213sin abandoned D6291: Allow to apply paint info to Polystar.

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

Jun 15 2018, 3:02 AM

Jun 11 2018

id213sin added a comment to D6257: edje: fix an issue "description.text.text" is not shown.

@herdsman
Thank you for quick review!

Jun 11 2018, 10:03 PM · efl
id213sin added a revision to T6997: Broken textblock.: D6257: edje: fix an issue "description.text.text" is not shown.
Jun 11 2018, 5:56 AM · efl: widgets, regression

Jun 4 2018

id213sin updated the diff for D5490: evas textblock: add/apply cursor cluster APIs based on grapheme cluster.

Remove an irrelevant change from test_entry.c

Jun 4 2018, 9:15 PM · efl
id213sin added a comment to D5490: evas textblock: add/apply cursor cluster APIs based on grapheme cluster.

@herdsman
Thank you for comment.

Jun 4 2018, 1:41 AM · efl
id213sin updated the diff for D5490: evas textblock: add/apply cursor cluster APIs based on grapheme cluster.

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.
Jun 4 2018, 1:32 AM · efl