Abdelrazak Younes wrote:
Dov Feldstern wrote:
Again, if this doesn't go in, RTL should be turned off, because that
*will* cause crashes without this!
Jose, as far as I see this patch seems OK but I didn't look at it very
closely nor tested it. JMarc will probably do a better job than me
reviewing it. If JMarc cannot review it before beta3, I suggest to
disable the RTL support by default for now.
Abdel.
The only thing (with turning RTL off again) is, as Darren (who already
ran into the crash due to this) pointed out, how are we going to find
out what problems there are with it if we don't leave it on? Wouldn't it
be better to keep it in, and only turn it off in the final release if we
still haven't been able to stabilize it? (For the same reason, I'd also
be happier if my patch went in, so that we get much more extensive
testing of it, as well...)
But I guess this depends on who exactly we see as the target audience of
the beta releases...
Dov
- Re: [patch] Re: Crash with RTL Dov Feldstern
-