Re: [Mlterm-dev-en] force slash character as LTR separator

[dhead f. <myf...@gm...>]

On Thu, Jun 26, 2014 at 1:43 PM, Araki Ken <ara...@us...> wrote:

>
>  Please try to apply the attached patch and start mlterm with
> --bimode=cmd_l
> option.
>

Thanks Araki for the quick resonse.
I'm not happy with the solution as the  --bimode=cmd_l affects the position
of a number character (e.g. "9") and also space (' ').

I got to another solution (see bellow) but I'm also not happy with it as
for example it doesn't give a proper attention to a folder/file with a mix
of RTL and LTR script's characters.
I think the best approach for me to continue is to dive into the code and
fix this myself.
In general, without reading through mlterm's code my idea for a solution
would be to make mlterm much more aware of the structure of the line.

>From 3bf21e0f36769c5b8b937adc3e00009aca9f07df Mon Sep 17 00:00:00 2001
From: dhead666 <myf...@gm...>
Date: Thu, 26 Jun 2014 13:25:53 +0300
Subject: [PATCH] force slash as ltr

---
 mlterm/libctl/ml_bidi.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/mlterm/libctl/ml_bidi.c b/mlterm/libctl/ml_bidi.c
index ddd232c..29ecbf7 100644
--- a/mlterm/libctl/ml_bidi.c
+++ b/mlterm/libctl/ml_bidi.c
@@ -147,6 +147,11 @@ ml_bidi(
                        /* Regarded as LTR character. */
                        fri_src[count] = 'a' ;
                }
+               else if ( code == '/' )
+               {
+                       /* Regarded as LTR character. */
+                       fri_src[count] = 'a' ;
+               }
                else
                {
                #ifdef  __DEBUG
--
2.0.0