Martin Vermeer wrote:
On Mon, Jun 25, 2007 at 11:17:15PM +0300, Dov Feldstern wrote:
Dov Feldstern wrote:
Hi!

Here's another RTL regression --- I think this should be really easy to solve for anyone familiar with the painting code. I'd be happy to try and help out if necessary.

http://bugzilla.lyx.org/show_bug.cgi?id=3889

Thanks!
Dov


Hi!

Attached find a patch for this bug. OK?

Twice almost the same code sequence. Any chance of factoring this out?

I know... But if you'll take a look at the two functions (paintText and cursorX), the loop at the heart of them is very similar, and a lot of the same things are happening in both functions, except they're written in different forms. But it is actually very very similar. I think that to correctly factor this out would mean rewriting the loop for both these functions, which is not something for now...

- Martin

Reply via email to