Re: [putty] Broken Bidi in PuTTY

[Nadim Shaikli <shaikli at yahoo dot com>]

--- Simon Tatham wrote:
> Nadim Shaikli wrote:
> > Simon, I just downloaded the "latest development snapshot" PuTTY
> > binary and it looks like its Bidi support is either broken or
> > disabled for some reason
> 
> I haven't done anything that _should_ have broken it, so if this is
> true then it's certainly a bug that needs fixing.

OK, then there definitely is a bug that wasn't there before :-)

> >  + 2004-12-02:r2941  Works great without issues
> >  + 2004-12-19:r5011  Seems to be broken
> 
> (Presumably you mean 2004-12-02:r4941 :-)

Yeah sorry - typo.

> > I simply looked at a mixed Arabic/English file and it looks like
> > the Arabic is being rendered properly its just not being flipped
> > so instead of reading HELLO, I'm seeing OLLEH (given caps are Arabic).
> 
> I'm afraid I can't reproduce this. 

I'm including a couple of temporary screenshots [1][2] to illustrate
the problem.  Do please note that there are a plethora of files in
Arabeyes' "translate" arm of its CVS [3].  I randomly selected a
file from Gnome [4] to view/compare.

> I've actually made one pass through the code myself fixing array
> overruns, which could easily be what broke it! :-) More eyes are
> always helpful, though.

Doubtful, but you never know.  The problem definitely manifests
itself in lines where Arabic/Latin are mixed.

BTW: my litmus test always includes a careful inspection of this
     ever-growing/changing file [5].

[1] http://arabeyes.org/~nadim/pic/putty/putty_good.png
[2] http://arabeyes.org/~nadim/pic/putty/putty_bad.png
[3] http://cvs.arabeyes.org/viewcvs/translate/
[4] http://cvs.arabeyes.org/viewcvs/translate/gnome/desktop/yelp.po
[5] http://arabeyes.org/~nadim/ar_utf8

Regards,

 - Nadim



		
__________________________________ 
Do you Yahoo!? 
Meet the all-new My Yahoo! - Try it today! 
http://my.yahoo.com