[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: PuTTY Testing
- To: developer at arabeyes dot org
- Subject: Re: PuTTY Testing
- From: "ahmad khalifa" <ahmadkhalifa at hotmail dot com>
- Date: Wed, 24 Mar 2004 09:05:45 +0000
From: Nadim Shaikli <shaikli at yahoo dot com>
Here are a few things I've encountered that I think should be fixed
before we push for the patch's adoption.
1. Do some repetitive 'ls -lF' and 'clear', putty crashed on me a
number of times (there might be a memory leak somewhere).
did u try that in an ordinary putty (without my patch) ??
i'll try to reproduce that crash.
2. If you view that favorite test file 'ar_utf8' there is a string
in there that notes, "Here are some Random words:" for whatever
reason the ":" dances around and changes location. 'cat' the
file (or less it) and then scroll back to that line (towards
the top to see what I mean).
hmm, you're really good at finding these things. maybe u shouldnt
be testing PuTTY. :)))
http://arabeyes.org/~nadim/ar_utf8
BTW: that line is not shown correctly (ie. I don't think Bidi
is doing the right thing there.
u know im a windows person, so on my Win2k, both Internet Explorer
and Netscape do some really strange Bidi on the ar_utf8 file. i think
my code does a much better job... so anyway, i dont know the exact
display of the line, would anyone _spell out_ the exact display of that
line ? because my IE and NS show some strange stuff...
3. The LAM+ALEF which gets combined into a LAA still exhibits a
space after it. That space should really be visually gone.
i could shift the rest of the line to override the space, but will this be
OK with the cursor and stuff?
4. Do please add 2 buttons in Terminal->Features that notes,
a. Disable Shaping
b. Disable Bidi
OK. but i'll keep them in a separate patch for now. just to be safe.
> anyway, i tested it and seem to get 100% Bidi Compliance,
> anyone care to correct me ?
The visual look of raw data looks semi-clean (note above).
I think once things are settled looking into Control-Characters
will be warranted in order to properly place cursors and/or string
search boxes, etc (that's for much later on).
i did run the tests, didnt u see the screen shots ?? i only added
mirroring code to it after the screen shots...
btw: i really hate the mirroring function, any hints there.?
btw2: i wrote those 2 small C programs to convert from
UnicodeData.txt into a big C array of character types. and another
one to write a very biig switch for mirroring.
i.e
1. takes UnicodeData.txt and outputs a file with lines like this...
/* 0000 */ ON, ON, L, R, NSM, RLE, RLE, NSM,
2. takes the mirroring file and outputs the following 2 lines for each entry
case 0028:
*ch = 0029;
break;
ak.
_________________________________________________________________
Help STOP SPAM with the new MSN 8 and get 2 months FREE*
http://join.msn.com/?page=features/junkmail