#ifdef INFINITE_CANVAS

Boudewijn Rempt boud at valdyas.org
Sat Dec 5 11:45:17 CET 2009


On Friday 04 December 2009, LukasT.dev at gmail.com wrote:
> Hi,
> 
> I commited fix for the outline of the selection by outline tool and I
>  wanted to ask what we will do with the buffer I added to QPainter canvas?
>  Should I still produce two versions code? One for the canvas with buffer
>  compiled and one without? I added black background to the dotted line as
>  second version btw.
> 
> When I added the QImage as another buffer there were some things about the
> speed. So do we plan to leave the buffer or remove? Or merge somehow with
> projection? But how? QImage in QPainter canvas = projection image + gray
> borders => RasterOp_XOR works for tools
> 
> The QImage is used so that every platform uses RasterOp for the canvas so
>  that we get RasterOp_XOR working and having visible tool outlines.
> 
> On the other hand it may be slowing down the performance because of another
> buffer. I did not notice any difference but I don't know how to measure it.
> 
> So what? Remove the ifdef and use INFINITE or produce two versions? That
>  means support 3 canvas types = OpenGL and QPainter with and without
>  RasterOp_XOR.
> 

I have to admit that I really haven't got a clue here :-(, so I cannot make 
any pronouncement ex cathedra about the issue.

-- 
Boudewijn Rempt | http://www.valdyas.org


More information about the kimageshop mailing list