[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: branches/work/kdepim-3.5.5+/korganizer/printing
From:       David Faure <dfaure () klaralvdalens-datakonsult ! se>
Date:       2006-12-11 17:01:02
Message-ID: 200612111802.46855.dfaure () klaralvdalens-datakonsult ! se
[Download RAW message or body]

On Monday 11 December 2006 17:40, Reinhold Kainhofer wrote:
> void CalPrintPluginBase::drawBox( QPainter &p, int linewidth, const QRect &rect )
> {
> -  QPen oldPen( p.pen() );
> -  QPen newPen( oldPen );
> -  newPen.setWidth( linewidth );
> -  p.setPen( newPen );
> +  p.save();
> +  QPen pen( p.pen() );
> +  pen.setWidth( linewidth );
> +  p.setPen( pen );
> p.drawRect( rect );
> -  p.setPen( oldPen );
> +  p.restore();
> }
Note that pen+setPen+setPen is much faster than a full save+restore, which might \
matter if this method is called often.

-- 
David Faure, faure@kde.org, dfaure@klaralvdalens-datakonsult.se
KDE/KOffice developer, Qt consultancy projects
Klarälvdalens Datakonsult AB, Platform-independent software solutions


[prev in list] [next in list] [prev in thread] [next in thread]