[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: PuTTY Testing






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