Line tool preview
Sven Langkamp
sven.langkamp at gmail.com
Fri Jul 10 16:11:40 CEST 2009
On Fri, Jul 10, 2009 at 4:00 PM, LukasT.dev at gmail.com
<lukast.dev at gmail.com>wrote:
> Hi,
> I noticed little stuff that can be done differently.
>
> Take a look at the method KisToolLine::paintLine(QPainter& gc, const
> QRect&)
> for painting line-preview when you use line tool:
> http://pastebin.com/m2cba0f11
>
> I think that instead of static_cast<int> we could use qRound as it is more
> precise. 0,7 is 0 with static_cast. With qRound 1.
>
> I'm not sure, that's why I'm asking.
>
> If you agree to use qRound, I can commit.
> Lukas
>
You could simply use QPointF instead. I think that is still from the Qt 3
times.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kimageshop/attachments/20090710/d2ef7cba/attachment.htm
More information about the kimageshop
mailing list