yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't have more than one reference to the same dummy variable in one

Browse source 
borders() call. So innocent decoration developers can be sure that
a value of one reference won't change a few lines later just because it
has been overwritten by another reverence to the same value...

svn path=/trunk/kdebase/kwin/; revision=299848
This commit is contained in:
Sandro Giessl 2004-03-30 13:52:27 +00:00
parent 186e42c151
commit c311b39bd3
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
kcmkwin/kwindecoration/preview.cpp
View file

@ -106,7 +106,8 @@ void KDecorationPreview::resizeEvent( QResizeEvent* e )
void KDecorationPreview::positionPreviews()
{
int titleBarHeight, leftBorder, rightBorder, xoffset, dummy;
int titleBarHeight, leftBorder, rightBorder, xoffset,
dummy1, dummy2, dummy3;
QRect geometry;
QSize size;
@ -115,8 +116,9 @@ void KDecorationPreview::positionPreviews()
if ( !deco[Active] || !deco[Inactive] )
return;
deco[Active]->borders( dummy, dummy, titleBarHeight, dummy );
deco[Inactive]->borders( leftBorder, rightBorder, dummy, dummy );
// don't have more than one reference to the same dummy variable in one borders() call.
deco[Active]->borders( dummy1, dummy2, titleBarHeight, dummy3 );
deco[Inactive]->borders( leftBorder, rightBorder, dummy1, dummy2 );
titleBarHeight = kMin( int( titleBarHeight * .9 ), 30 );
xoffset = kMin( kMax( 10, QApplication::reverseLayout()