Merge lp:~mc-return/compiz/compiz.merge-fix1166195-fix1166196-fix116245-resizeinfo-fixes into lp:compiz/0.9.10

[MC Return]

*Resizeinfo, xml changes:

Added option for bold/normal font, default is still bold.

Added option to change the font size (10-14), default is still 12
pixel.

Enhanced and corrected a few tooltips.

*Resizeinfo, code changes:

Choose between PANGO_WEIGHT_BOLD and PANGO_WEIGHT_NORMAL.

Use individual font size specified by the user in CCSM.

Fixed computation of wrong damageRegion, it was additionally adding
the window size, making it way too large.

Removed useless declaration of int width and height.

Declaration and assignment of local variables in one line, if possible.

Minor indentation fixes.

(LP: #1166195, LP: #1166196 and LP: #1166245)

[Sam Spilsbury (smspillaz)]

Hi,

Please see my previous comments on the usage of the boy-scout rule.

These were the only changes that were actually substantial:

77 -const unsigned short RESIZE_POPUP_WIDTH = 85;
78 -const unsigned short RESIZE_POPUP_HEIGHT = 50;
79 +const unsigned short RESIZE_POPUP_WIDTH = 100;
80 +const unsigned short RESIZE_POPUP_HEIGHT = 33;

149 pango_font_description_set_family (font,"Sans");
150 - pango_font_description_set_absolute_size (font, 12 * PANGO_SCALE);
151 + pango_font_description_set_absolute_size (font,
152 + is->optionGetResizeinfoFontSize () *
153 + PANGO_SCALE);
154 pango_font_description_set_style (font, PANGO_STYLE_NORMAL);
155 - pango_font_description_set_weight (font, PANGO_WEIGHT_BOLD);
156 -
157 +
158 + if (is->optionGetResizeinfoFontBold ())
159 + pango_font_description_set_weight (font, PANGO_WEIGHT_BOLD);
160 + else
161 + pango_font_description_set_weight (font, PANGO_WEIGHT_NORMAL);

As for those changes, they seem fine to me. I haven't got time to look at the formatting things just yet though.
162 +

[MC Return (mc-return)]

>
> These were the only changes that were actually substantial:
>
> 77 -const unsigned short RESIZE_POPUP_WIDTH = 85;
> 78 -const unsigned short RESIZE_POPUP_HEIGHT = 50;
> 79 +const unsigned short RESIZE_POPUP_WIDTH = 100;
> 80 +const unsigned short RESIZE_POPUP_HEIGHT = 33;
>
> 149 pango_font_description_set_family (font,"Sans");
> 150 - pango_font_description_set_absolute_size (font, 12 * PANGO_SCALE);
> 151 + pango_font_description_set_absolute_size (font,
> 152 + is->optionGetResizeinfoFontSize () *
> 153 + PANGO_SCALE);
> 154 pango_font_description_set_style (font, PANGO_STYLE_NORMAL);
> 155 - pango_font_description_set_weight (font, PANGO_WEIGHT_BOLD);
> 156 -
> 157 +
> 158 + if (is->optionGetResizeinfoFontBold ())
> 159 + pango_font_description_set_weight (font, PANGO_WEIGHT_BOLD);
> 160 + else
> 161 + pango_font_description_set_weight (font, PANGO_WEIGHT_NORMAL);
>
> As for those changes, they seem fine to me. I haven't got time to look at the
> formatting things just yet though.
> 162 +

TBH, the most substantial change in this MP was not listed by you ;)
It is this:

241 - (x + RESIZE_POPUP_WIDTH + 5),
242 - (y + RESIZE_POPUP_HEIGHT + 5));
243 + (RESIZE_POPUP_WIDTH + 5),
244 + (RESIZE_POPUP_HEIGHT + 5));

It reduces the damage rectangle's size by the window size...

[MC Return (mc-return)]

This does not mean anything for someone reading the tooltip:

<_long>Show resize info for all windows as opposed to just for windows with a resize increment of greater than 1.</_long>

so I changed it to:

<_long>Show resize info for all windows as opposed to just for text based windows like terminals.</_long>

The other changes in the tooltips are:
If the user adjusts the color, he can also adjust the opacity.

In the code I did some minor indentation fixes, I am sorry I can't let them untouched if I notice those...
Other than that and the things already mentioned by yourself, I just changed assignment and declaration of
local variables to be done in the same line, if possible...

[Sam Spilsbury (smspillaz)]

> This does not mean anything for someone reading the tooltip:
>
> <_long>Show resize info for all windows as opposed to just for windows with a
> resize increment of greater than 1.</_long>
>
> so I changed it to:
>
> <_long>Show resize info for all windows as opposed to just for text based
> windows like terminals.</_long>
>
> The other changes in the tooltips are:
> If the user adjusts the color, he can also adjust the opacity.
>
> In the code I did some minor indentation fixes, I am sorry I can't let them
> untouched if I notice those...
> Other than that and the things already mentioned by yourself, I just changed
> assignment and declaration of
> local variables to be done in the same line, if possible...

Indentation fixes and code cleanups are useful and appreciated. However, they tend to just confuse things when mixed them with substantive changes. Especially when the reviewer isn't immediately familiar with the section of code that's being edited.

I think this guideline should be followed when making such changes:
 1. If making a substantive change (eg, a change in behaviour, fixing the logic or a bug) keep the formatting, stylistic and other minor changes restricted to the code section in which the application logic is being changed. Do not change formatting, indentation, variable declarations or perform any other refactoring in unrelated code sections in the same, or other files.
 2. Merge proposals which change only indentation, formatting, variable declarations and perform other refactoring are 100% acceptable and encouraged (so long as they improve the quality of the code), however, they must be marked as separate so that reviewers know what to look out for.

Doing changes in this manner means that reviewers can get through both sets faster, because the type of cognitive load is different for each. In addition, it means that substantive changes aren't blocked on formatting changes which other reviewers might have suggestions on.

[Sam Spilsbury (smspillaz)]

>
> TBH, the most substantial change in this MP was not listed by you ;)
> It is this:
>
> 241 - (x + RESIZE_POPUP_WIDTH + 5),
> 242 - (y + RESIZE_POPUP_HEIGHT + 5));
> 243 + (RESIZE_POPUP_WIDTH + 5),
> 244 + (RESIZE_POPUP_HEIGHT + 5));
>
> It reduces the damage rectangle's size by the window size...

Right, this is exactly the problem I'm referring to. Mixing behavioural and formatting changes can cause important things like this to be missed.

[Sam Spilsbury (smspillaz)]

179 - int height = RESIZE_POPUP_HEIGHT;
180 - int width = RESIZE_POPUP_WIDTH;
181 float r, g, b, a;
182
183 INFO_SCREEN (screen);
184 @@ -197,7 +199,9 @@
185 cairo_set_operator (cr, CAIRO_OPERATOR_OVER);
186
187 /* Setup Gradient */
188 - pattern = cairo_pattern_create_linear (0, 0, width, height);
189 + pattern = cairo_pattern_create_linear (0, 0,
190 + RESIZE_POPUP_WIDTH,
191 + RESIZE_POPUP_HEIGHT);
192
193 r = is->optionGetGradient1Red () / (float)0xffff;
194 g = is->optionGetGradient1Green () / (float)0xffff;
195 @@ -220,10 +224,14 @@
196
197 /* Rounded Rectangle! */
198 cairo_arc (cr, border, border, border, PI, 1.5f * PI);
199 - cairo_arc (cr, border + width - 2 * border, border, border,
200 + cairo_arc (cr, border + RESIZE_POPUP_WIDTH - 2 * border,
201 + border, border,
202 1.5f * PI, 2.0 * PI);
203 - cairo_arc (cr, width - border, height - border, border, 0, PI / 2.0f);
204 - cairo_arc (cr, border, height - border, border, PI / 2.0f, PI);
205 + cairo_arc (cr, RESIZE_POPUP_WIDTH - border,
206 + RESIZE_POPUP_HEIGHT - border,
207 + border, 0, PI / 2.0f);
208 + cairo_arc (cr, border, RESIZE_POPUP_HEIGHT - border,
209 + border, PI / 2.0f, PI);

Instead of removing the declarations of width and height, it might be better to change their purpose. We seem to be doing a lot of this:

RESIZE_POPUP_WIDTH - border ... RESIZE_POPUP_HEIGHT - border etc

Why not change the declaration to

unsigned int widthWithoutBorder = RESIZE_POPUP_WIDTH - border;
unsigned int heightWithoutBorder = RESIZE_POPUP_HEIGHT - border;

Then you can just do:

cairo_arc (cr, widthWithoutBorder,
206 + heightWithoutBorder,
207 + border, 0, PI / 2.0f);

etc

[MC Return (mc-return)]

>
> Indentation fixes and code cleanups are useful and appreciated. However, they
> tend to just confuse things when mixed them with substantive changes.
> Especially when the reviewer isn't immediately familiar with the section of
> code that's being edited.
>
Yes, you are right - I apologize once again.

> I think this guideline should be followed when making such changes:
> 1. If making a substantive change (eg, a change in behaviour, fixing the
> logic or a bug) keep the formatting, stylistic and other minor changes
> restricted to the code section in which the application logic is being
> changed. Do not change formatting, indentation, variable declarations or
> perform any other refactoring in unrelated code sections in the same, or other
> files.
> 2. Merge proposals which change only indentation, formatting, variable
> declarations and perform other refactoring are 100% acceptable and encouraged
> (so long as they improve the quality of the code), however, they must be
> marked as separate so that reviewers know what to look out for.
>
Very good suggestion. I will do it that way from now on.

First I'll do a merge proposal with just substantial changes, then a follow-up
MP cleaning up that file's indentation and formatting problems only...

> Doing changes in this manner means that reviewers can get through both sets
> faster, because the type of cognitive load is different for each. In addition,
> it means that substantive changes aren't blocked on formatting changes which
> other reviewers might have suggestions on.

Agreed. I will try my best ;)

[MC Return (mc-return)]

> 179 - int height = RESIZE_POPUP_HEIGHT;
> 180 - int width = RESIZE_POPUP_WIDTH;
> 181 float r, g, b, a;
> 182
> 183 INFO_SCREEN (screen);
> 184 @@ -197,7 +199,9 @@
> 185 cairo_set_operator (cr, CAIRO_OPERATOR_OVER);
> 186
> 187 /* Setup Gradient */
> 188 - pattern = cairo_pattern_create_linear (0, 0, width, height);
> 189 + pattern = cairo_pattern_create_linear (0, 0,
> 190 + RESIZE_POPUP_WIDTH,
> 191 + RESIZE_POPUP_HEIGHT);
> 192
> 193 r = is->optionGetGradient1Red () / (float)0xffff;
> 194 g = is->optionGetGradient1Green () / (float)0xffff;
> 195 @@ -220,10 +224,14 @@
> 196
> 197 /* Rounded Rectangle! */
> 198 cairo_arc (cr, border, border, border, PI, 1.5f * PI);
> 199 - cairo_arc (cr, border + width - 2 * border, border, border,
> 200 + cairo_arc (cr, border + RESIZE_POPUP_WIDTH - 2 * border,
> 201 + border, border,
> 202 1.5f * PI, 2.0 * PI);
> 203 - cairo_arc (cr, width - border, height - border, border, 0, PI /
> 2.0f);
> 204 - cairo_arc (cr, border, height - border, border, PI / 2.0f, PI);
> 205 + cairo_arc (cr, RESIZE_POPUP_WIDTH - border,
> 206 + RESIZE_POPUP_HEIGHT - border,
> 207 + border, 0, PI / 2.0f);
> 208 + cairo_arc (cr, border, RESIZE_POPUP_HEIGHT - border,
> 209 + border, PI / 2.0f, PI);
>
> Instead of removing the declarations of width and height, it might be better
> to change their purpose. We seem to be doing a lot of this:
>
> RESIZE_POPUP_WIDTH - border ... RESIZE_POPUP_HEIGHT - border etc
>
> Why not change the declaration to
>
> unsigned int widthWithoutBorder = RESIZE_POPUP_WIDTH - border;
> unsigned int heightWithoutBorder = RESIZE_POPUP_HEIGHT - border;
>
> Then you can just do:
>
> cairo_arc (cr, widthWithoutBorder,
> 206 + heightWithoutBorder,
> 207 + border, 0, PI / 2.0f);
>
> etc

Yes, you are right. That would be a possibility.
But
"RESIZE_POPUP_WIDTH - border" is just used once and
"RESIZE_POPUP_HEIGHT - border" is just used twice in the code

So does this really justify creating 2 new variables ?
I do not think I agree here...

[Sam Spilsbury (smspillaz)]

On Tue, Apr 9, 2013 at 5:24 PM, MC Return <email address hidden> wrote:
>>
>> Indentation fixes and code cleanups are useful and appreciated. However, they
>> tend to just confuse things when mixed them with substantive changes.
>> Especially when the reviewer isn't immediately familiar with the section of
>> code that's being edited.
>>
> Yes, you are right - I apologize once again.

No need to apologize, this was merely an observation.

>
>> I think this guideline should be followed when making such changes:
>> 1. If making a substantive change (eg, a change in behaviour, fixing the
>> logic or a bug) keep the formatting, stylistic and other minor changes
>> restricted to the code section in which the application logic is being
>> changed. Do not change formatting, indentation, variable declarations or
>> perform any other refactoring in unrelated code sections in the same, or other
>> files.
>> 2. Merge proposals which change only indentation, formatting, variable
>> declarations and perform other refactoring are 100% acceptable and encouraged
>> (so long as they improve the quality of the code), however, they must be
>> marked as separate so that reviewers know what to look out for.
>>
> Very good suggestion. I will do it that way from now on.
>
> First I'll do a merge proposal with just substantial changes, then a follow-up
> MP cleaning up that file's indentation and formatting problems only..

That makes sense, thanks :)
.
>
>> Doing changes in this manner means that reviewers can get through both sets
>> faster, because the type of cognitive load is different for each. In addition,
>> it means that substantive changes aren't blocked on formatting changes which
>> other reviewers might have suggestions on.
>
> Agreed. I will try my best ;)

> "RESIZE_POPUP_WIDTH - border" is just used once and
> "RESIZE_POPUP_HEIGHT - border" is just used twice in the code
> So does this really justify creating 2 new variables ?

My rule of thumb tends to be more extreme than the prevailing view,
which is that if you have some value that is being used as a value
passed to a function, then its a good idea to keep it in a constant if
the calculation is done more than once, as opposed to doing the
calculation multiple times as nested expressions within its
parameters. Its just a minor thing though, so its not something that
bothers me a whole lot.

> --
> https://code.launchpad.net/~mc-return/compiz/compiz.merge-fix1166195-fix1166196-fix116245-resizeinfo-fixes/+merge/157670
> Your team Compiz Maintainers is requested to review the proposed merge of lp:~mc-return/compiz/compiz.merge-fix1166195-fix1166196-fix116245-resizeinfo-fixes into lp:compiz.

--
Sam Spilsbury

[Sam Spilsbury (smspillaz)]

review: Approve

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

=== modified file 'plugins/resizeinfo/resizeinfo.xml.in'
--- plugins/resizeinfo/resizeinfo.xml.in	2012-11-06 10:20:30 +0000
+++ plugins/resizeinfo/resizeinfo.xml.in	2013-04-08 15:17:24 +0000
@@ -24,13 +24,25 @@
 		<max>5000</max>
 	    </option>
 	    <option name="always_show" type="bool">
-		<_short>Show resize info for all windows</_short>
-		<_long>Show resize info for all windows as opposed to just for windows with a resize increment of greater than 1.</_long>
+		<_short>Show Resizeinfo For All Windows</_short>
+		<_long>Show resize info for all windows as opposed to just for text based windows like terminals.</_long>
 		<default>false</default>
 	    </option>
+	    <option name="resizeinfo_font_bold" type="bool">
+		<_short>Bold Font</_short>
+		<_long>Render the resize info's size display with a bold font.</_long>
+		<default>true</default>
+	    </option>
+	    <option name="resizeinfo_font_size" type="int">
+		<_short>Font Size</_short>
+		<_long>Font size of the resize info's display.</_long>
+		<default>12</default>
+		<min>10</min>
+		<max>14</max>
+	    </option>
 	    <option name="text_color" type="color">
 		<_short>Text Color</_short>
-		<_long>Color of the resize popup's text.</_long>
+		<_long>Color and opacity of the resize popup's text.</_long>
 		<default>
 		    <red>0x0000</red>
 		    <blue>0x0000</blue>
@@ -40,7 +52,7 @@
 	    </option>
 	    <option name="gradient_1" type="color">
 		<_short>Gradient Color 1</_short>
-		<_long>Color on the left side of the gradient background.</_long>
+		<_long>Color and opacity on the left side of the gradient background.</_long>
 		<default>
 		    <red>0xcccc</red>
 		    <green>0xcccc</green>
@@ -50,7 +62,7 @@
 	    </option>
 	    <option name="gradient_2" type="color">
 		<_short>Gradient Color 2</_short>
-		<_long>Color in the center of the gradient background.</_long>
+		<_long>Color and opacity in the center of the gradient background.</_long>
 		<default>
 		    <red>0xf332</red>
 		    <green>0xf332</green>
@@ -60,7 +72,7 @@
 	    </option>
 	    <option name="gradient_3" type="color">
 		<_short>Gradient Color 3</_short>
-		<_long>Color on the right side of the gradient background.</_long>
+		<_long>Color and opacity on the right side of the gradient background.</_long>
 		<default>
 		    <red>0xd998</red>
 		    <green>0xd998</green>
@@ -70,7 +82,7 @@
 	    </option>
 	    <option name="outline_color" type="color">
 		<_short>Outline Color</_short>
-		<_long>Color of the background's outline.</_long>
+		<_long>Color and opacity of the background's outline.</_long>
 		<default>
 		    <red>0xe665</red>
 		    <green>0xe665</green>

=== modified file 'plugins/resizeinfo/src/resizeinfo.cpp'
--- plugins/resizeinfo/src/resizeinfo.cpp	2013-02-27 11:33:10 +0000
+++ plugins/resizeinfo/src/resizeinfo.cpp	2013-04-08 15:17:24 +0000
@@ -25,8 +25,8 @@
 
 COMPIZ_PLUGIN_20090315 (resizeinfo, InfoPluginVTable);
 
-const unsigned short RESIZE_POPUP_WIDTH = 85;
-const unsigned short RESIZE_POPUP_HEIGHT = 50;
+const unsigned short RESIZE_POPUP_WIDTH = 100;
+const unsigned short RESIZE_POPUP_HEIGHT = 33;
 
 const double PI = 3.14159265359f;
 
@@ -57,16 +57,17 @@
 	return;
 
     pixmap = XCreatePixmap (screen->dpy (), screen->root (),
-			    RESIZE_POPUP_WIDTH, RESIZE_POPUP_HEIGHT, 32);
+			    RESIZE_POPUP_WIDTH,
+			    RESIZE_POPUP_HEIGHT, 32);
     if (!pixmap)
 	return;
 
-    surface =
-	cairo_xlib_surface_create_with_xrender_format (screen->dpy (),
-						       pixmap, s,
-						       format,
-						       RESIZE_POPUP_WIDTH,
-						       RESIZE_POPUP_HEIGHT);
+    surface = cairo_xlib_surface_create_with_xrender_format (screen->dpy (),
+							     pixmap, s,
+							     format,
+							     RESIZE_POPUP_WIDTH,
+							     RESIZE_POPUP_HEIGHT);
+
     if (cairo_surface_status (surface) != CAIRO_STATUS_SUCCESS)
     {
 	compLogMessage ("resizeinfo", CompLogLevelWarn,
@@ -102,9 +103,6 @@
 void
 InfoLayer::renderText ()
 {
-    unsigned int         baseWidth, baseHeight;
-    unsigned int         widthInc, heightInc;
-    unsigned int         width, height, xv, yv;
     unsigned short       *color;
     char                 info[50];
     PangoLayout          *layout;
@@ -116,17 +114,17 @@
     if (!valid)
 	return;
 
-    baseWidth = is->pWindow->sizeHints ().base_width;
-    baseHeight = is->pWindow->sizeHints ().base_height;
-    widthInc = is->pWindow->sizeHints ().width_inc;
-    heightInc = is->pWindow->sizeHints ().height_inc;
-    width = is->resizeGeometry.width;
-    height = is->resizeGeometry.height;
-	
+    unsigned int baseWidth  = is->pWindow->sizeHints ().base_width;
+    unsigned int baseHeight = is->pWindow->sizeHints ().base_height;
+    unsigned int widthInc   = is->pWindow->sizeHints ().width_inc;
+    unsigned int heightInc  = is->pWindow->sizeHints ().height_inc;
+    unsigned int width      = is->resizeGeometry.width;
+    unsigned int height     = is->resizeGeometry.height;
+
     color = is->optionGetTextColor ();
 
-    xv = (widthInc > 1) ? (width - baseWidth) / widthInc : width;
-    yv = (heightInc > 1) ? (height - baseHeight) / heightInc : height;
+    unsigned int xv = (widthInc > 1) ? (width - baseWidth) / widthInc : width;
+    unsigned int yv = (heightInc > 1) ? (height - baseHeight) / heightInc : height;
 
     /* Clear the context. */
     cairo_save (cr);
@@ -141,10 +139,16 @@
     layout = pango_cairo_create_layout (is->textLayer.cr);
   
     pango_font_description_set_family (font,"Sans");
-    pango_font_description_set_absolute_size (font, 12 * PANGO_SCALE);
+    pango_font_description_set_absolute_size (font,
+					      is->optionGetResizeinfoFontSize () *
+					      PANGO_SCALE);
     pango_font_description_set_style (font, PANGO_STYLE_NORMAL);
-    pango_font_description_set_weight (font, PANGO_WEIGHT_BOLD);
- 
+
+    if (is->optionGetResizeinfoFontBold ())
+	pango_font_description_set_weight (font, PANGO_WEIGHT_BOLD);
+    else
+	pango_font_description_set_weight (font, PANGO_WEIGHT_NORMAL);
+
     pango_layout_set_font_description (layout, font);
     pango_layout_set_ellipsize (layout, PANGO_ELLIPSIZE_END);
     pango_layout_set_text (layout, info, -1);
@@ -152,7 +156,7 @@
     pango_layout_get_pixel_size (layout, &w, &h);
   
     cairo_move_to (cr, 
-		   RESIZE_POPUP_WIDTH / 2.0f - w / 2.0f, 
+		   RESIZE_POPUP_WIDTH / 2.0f - w / 2.0f,
 		   RESIZE_POPUP_HEIGHT / 2.0f - h / 2.0f);
   
     pango_layout_set_width (layout, RESIZE_POPUP_WIDTH * PANGO_SCALE);
@@ -178,8 +182,6 @@
 {
     cairo_pattern_t *pattern;	
     float           border = 7.5;
-    int             height = RESIZE_POPUP_HEIGHT;
-    int             width = RESIZE_POPUP_WIDTH;
     float           r, g, b, a;
 
     INFO_SCREEN (screen);
@@ -197,7 +199,9 @@
     cairo_set_operator (cr, CAIRO_OPERATOR_OVER);
 
     /* Setup Gradient */
-    pattern = cairo_pattern_create_linear (0, 0, width, height);
+    pattern = cairo_pattern_create_linear (0, 0,
+					   RESIZE_POPUP_WIDTH,
+					   RESIZE_POPUP_HEIGHT);
 
     r = is->optionGetGradient1Red () / (float)0xffff;
     g = is->optionGetGradient1Green () / (float)0xffff;
@@ -220,10 +224,14 @@
 	
     /* Rounded Rectangle! */
     cairo_arc (cr, border, border, border, PI, 1.5f * PI);
-    cairo_arc (cr, border + width - 2 * border, border, border,
+    cairo_arc (cr, border + RESIZE_POPUP_WIDTH - 2 * border,
+	       border, border,
 	       1.5f * PI, 2.0 * PI);
-    cairo_arc (cr, width - border, height - border, border, 0, PI / 2.0f);
-    cairo_arc (cr, border, height - border, border,  PI / 2.0f, PI);
+    cairo_arc (cr, RESIZE_POPUP_WIDTH - border,
+	       RESIZE_POPUP_HEIGHT - border,
+	       border, 0, PI / 2.0f);
+    cairo_arc (cr, border, RESIZE_POPUP_HEIGHT - border,
+	       border,  PI / 2.0f, PI);
     cairo_close_path (cr);
     cairo_fill_preserve (cr);
 	
@@ -240,7 +248,7 @@
 
 static void
 backgroundColorChanged (CompOption                 *o, 
-		 ResizeinfoOptions::Options num)
+			ResizeinfoOptions::Options num)
 {
     INFO_SCREEN (screen);
 
@@ -250,19 +258,17 @@
 void
 InfoScreen::damagePaintRegion ()
 {
-    int    x, y;
-
     if (!fadeTime && !drawing)
 	return;
 
-    x = resizeGeometry.x + resizeGeometry.width / 2.0f -
-	RESIZE_POPUP_WIDTH / 2.0f;
-    y = resizeGeometry.y + resizeGeometry.height / 2.0f - 
-	RESIZE_POPUP_HEIGHT / 2.0f;
+    int x = resizeGeometry.x + resizeGeometry.width / 2.0f -
+	    RESIZE_POPUP_WIDTH / 2.0f;
+    int y = resizeGeometry.y + resizeGeometry.height / 2.0f -
+	    RESIZE_POPUP_HEIGHT / 2.0f;
 
     CompRegion reg (x - 5, y - 5,
-		    (x + RESIZE_POPUP_WIDTH + 5),
-		    (y + RESIZE_POPUP_HEIGHT + 5));
+		    (RESIZE_POPUP_WIDTH + 5),
+		    (RESIZE_POPUP_HEIGHT + 5));
 
     cScreen->damageRegion (reg);
 }
@@ -312,12 +318,11 @@
     INFO_SCREEN (screen);
 
     if ((!is->pWindow || !is->drawing) &&
-        ((window->state () & MAXIMIZE_STATE) != MAXIMIZE_STATE))
+	((window->state () & MAXIMIZE_STATE) != MAXIMIZE_STATE))
     {
-	bool showInfo;
-	showInfo = (((window->sizeHints ().width_inc != 1) && 
-		     (window->sizeHints ().height_inc != 1)) ||
-		    is->optionGetAlwaysShow ());
+	bool showInfo = (((window->sizeHints ().width_inc != 1) &&
+			 (window->sizeHints ().height_inc != 1)) ||
+			 is->optionGetAlwaysShow ());
 
 	if (showInfo && (mask & CompWindowGrabResizeMask))
 	{
@@ -333,7 +338,7 @@
 	    screen->handleEventSetEnabled (is, true);
 	}
     }
-	
+
     window->grabNotify (x, y, state, mask);
 }
 
@@ -359,8 +364,8 @@
    RESIZE_POPUP_HEIGHT with the opacity in InfoScreen. */
 void
 InfoLayer::draw (const GLMatrix &transform,
-                 int             x,
-	   	 int y)
+		 int             x,
+		 int             y)
 {
     BOX   box;
     float opacity;
@@ -375,9 +380,9 @@
 	GLushort           colorData[4];
 	GLfloat            textureData[8];
 	GLfloat            vertexData[12];
-	GLTexture         *tex = texture[i];
-	GLTexture::Matrix matrix = tex->matrix ();
-	GLVertexBuffer *streamingBuffer = GLVertexBuffer::streamingBuffer ();
+	GLTexture          *tex = texture[i];
+	GLTexture::Matrix  matrix = tex->matrix ();
+	GLVertexBuffer     *streamingBuffer = GLVertexBuffer::streamingBuffer ();
 
 	tex->enable (GLTexture::Good);
 
@@ -440,19 +445,16 @@
 			   CompOutput                *output,
 			   unsigned int              mask)
 {
-    bool status;
-  
-    status = gScreen->glPaintOutput (attrib, transform, region, output, mask);
+    bool status = gScreen->glPaintOutput (attrib, transform, region, output, mask);
 
     if ((drawing || fadeTime) && pWindow)
     {
 	GLMatrix sTransform = transform;
-	int      x, y;
 
-	x = resizeGeometry.x + resizeGeometry.width / 2.0f - 
-	    RESIZE_POPUP_WIDTH / 2.0f;
-	y = resizeGeometry.y + resizeGeometry.height / 2.0f - 
-	    RESIZE_POPUP_HEIGHT / 2.0f;
+	int x = resizeGeometry.x + resizeGeometry.width / 2.0f -
+		RESIZE_POPUP_WIDTH / 2.0f;
+	int y = resizeGeometry.y + resizeGeometry.height / 2.0f -
+		RESIZE_POPUP_HEIGHT / 2.0f;
 
 	sTransform.toScreenSpace (output, -DEFAULT_Z_CAMERA);
       
@@ -465,7 +467,6 @@
   
 	gScreen->setTexEnvMode (GL_REPLACE);
 	glDisable (GL_BLEND);
-
     }
 
     return status;
@@ -474,13 +475,15 @@
 void
 InfoScreen::handleEvent (XEvent *event)
 {
-    switch (event->type) {
-    case ClientMessage:    
+    switch (event->type)
+    {
+    case ClientMessage:
 	if (event->xclient.message_type == resizeInfoAtom)
 	{
 	    CompWindow *w;
 
 	    w = screen->findWindow (event->xclient.window);
+
 	    if (w && w == pWindow)
 	    {
 		resizeGeometry.x      = event->xclient.data.l[0];
@@ -500,6 +503,7 @@
 	    }
 	}
 	break;
+
     default:
 	break;
     }

=== modified file 'plugins/resizeinfo/src/resizeinfo.h'
--- plugins/resizeinfo/src/resizeinfo.h	2012-09-07 23:29:42 +0000
+++ plugins/resizeinfo/src/resizeinfo.h	2013-04-08 15:17:24 +0000
@@ -58,8 +58,8 @@
 	cairo_t           *cr;
 
 	void draw (const GLMatrix &transform,
-	           int             x,
-	 	   int y);
+		   int            x,
+		   int            y);
 
 	void renderBackground ();
 	void renderText ();
@@ -123,9 +123,9 @@
 
 	void
 	grabNotify (int,
-	      	    int,
-	      	    unsigned int,
-	      	    unsigned int);
+		    int,
+		    unsigned int,
+		    unsigned int);
 
 	void
 	ungrabNotify ();