Re: [dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-08 Thread Ryusei Yamaguchi
Hi. On 2016/03/08 21:48, Christoph Lohmann wrote: Thanks for changing the patch. I have applied it to mainline. Please still consider changing libxft or fontconfig or contributing to dtext for a suckless font rendering library. Ok. I'll make an effort. Thanks, Ryusei

Re: [dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-08 Thread Christoph Lohmann
Greetings. On Tue, 08 Mar 2016 13:48:59 +0100 Ryusei Yamaguchi wrote: > This fix is needed to use dual-width fonts, which have double-width > glyphs (e.g. CJK unified ideographs). Thanks for changing the patch. I have applied it to mainline. Please still consider changing libxft or fontconfi

Re: [dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-08 Thread Silvan Jegen
Hi On Tue, Mar 8, 2016 at 4:26 AM, Ryusei Yamaguchi wrote: > This fix is needed to use dual-width fonts, which have double-width > glyphs (e.g. CJK unified ideographs). > > Signed-off-by: Ryusei Yamaguchi > --- > config.def.h | 8 > st.c | 8 +++- > 2 files changed, 15 inse

[dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-07 Thread Ryusei Yamaguchi
This fix is needed to use dual-width fonts, which have double-width glyphs (e.g. CJK unified ideographs). Signed-off-by: Ryusei Yamaguchi --- config.def.h | 8 st.c | 8 +++- 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/config.def.h b/config.def.h index fd0

Re: [dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-07 Thread Christoph Lohmann
Greetings. On Mon, 07 Mar 2016 20:52:30 +0100 Ryusei Yamaguchi wrote: > On 2016/03/07 22:53, Christoph Lohmann wrote: > > Please give me more context, like a screenshot before and after and why this > > can't be fixed in fontconfig itself. This would be yet another fontconfig > > API > > hack in

Re: [dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-07 Thread Ryusei Yamaguchi
On 2016/03/07 22:53, Christoph Lohmann wrote: Please give me more context, like a screenshot before and after and why this can't be fixed in fontconfig itself. This would be yet another fontconfig API hack in st. Step to Reproduce: 1. Install M+ 1m from https://osdn.jp/projects/mplus-fonts/re

Re: [dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-07 Thread Silvan Jegen
Heyhey On Mon, Mar 7, 2016 at 7:06 AM, Ryusei Yamaguchi wrote: > This fix is needed to use dual-width fonts, which have double-width > glyphs (e.g. CJK unified ideographs). > > Signed-off-by: Ryusei Yamaguchi > --- > st.c | 15 ++- > 1 file changed, 14 insertions(+), 1 deletion(-)

Re: [dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-07 Thread Christoph Lohmann
Greetings. On Mon, 07 Mar 2016 14:53:50 +0100 Ryusei Yamaguchi wrote: > This fix is needed to use dual-width fonts, which have double-width > glyphs (e.g. CJK unified ideographs). Please give me more context, like a screenshot before and after and why this can't be fixed in fontconfig itself. Th

[dev] [PATCH] Measure the single advance width with a heuristic method

2016-03-06 Thread Ryusei Yamaguchi
This fix is needed to use dual-width fonts, which have double-width glyphs (e.g. CJK unified ideographs). Signed-off-by: Ryusei Yamaguchi --- st.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/st.c b/st.c index 0536b6f..fc49e67 100644 --- a/st.c +++ b/st.c @