~ivenvd/compiz/compiz.fix_1189374 : changes

~ivenvd/compiz/compiz.fix_1189374 » Changes from revision 3737

From Revision 3727 to 3708

Rev	Summary	Authors	Date
3727	Opacify code cleanup: Opacify code cleanup: Declaration of local variables outside of loops. Calculate activeOpacity and targetOpacity just once, not for each window. Use prefix instead of postfix increments. Merged if condition checks. Declaration and assignment of local variables in one line. Added and removed newlines. Removed redundant brackets. Fixed indentation. Opacify, functionality fix: Improved the toggling logic by setting isToggle in the constructor according to the startup setting, then calling setFunctions (isToggle);,which then calls screen->handleEventSetEnabled (os, os->isToggle); to fix Opacify ignoring the "Toggle Opacify on by default" setting. (LP: #787814). Fixes: https://bugs.launchpad.net/bugs/787814. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3726	Fade plugin, minor code cleanup: Fade plugin, minor code cleanup: Declaration and assignment of local variables in one line. Merged if condition checks. Use pre- instead of postfix de- and increments. Removed redundant brackets. Added and removed newlines. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3725	Fix memory leaks in titleinfo plugin. Fixes: https://bugs.la... Fix memory leaks in titleinfo plugin. Fixes: https://bugs.launchpad.net/bugs/1101454, https://bugs.launchpad.net/bugs/1101569. Approved by PS Jenkins bot, Sam Spilsbury.	Sami Jaktholm	11 years ago
3724	Rotate Cube code cleanup: Rotate Cube code cleanup: Simplified a few calculations of this type: (360.0 / (x * 2.0)) == 180 / x Declaration and assignment of variables in one line. Use pre- instead of postfix de- and increments. Merged if condition checks. Added and removed newlines and brackets. Fixed indentation. Added TODO comments. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3723	action.cpp and actions.cpp code cleanup: action.cpp and actions.cpp code cleanup: Use pre- instead of postfix increment. Declaration and assignment of variables in one line. Merged if condition checks. Removed redundant brackets. Added and removed newlines. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3722	Grid: Fix window misplacement if a window is restored but th... Grid: Fix window misplacement if a window is restored but the size doesn't change. - Set isGridResized to false before configuring the window in GridScreen::restoreWindow. If the restored geometry is the same as current geometry, configureXWindow will trigger moveNotify instead of resizeNotify. As isGridResized is still true, GridWindow::moveNotify blocks the restoration movement and the pointer detaches from the decoration. - Don't use pointerBufs when calculating position of the restored window with snapback windows disabled. The window will be positioned a bit too high (or low) and the pointer detaches from decoration if the movement was actually tracked. This was a workaround for the first point and can be safely removed. Fixes: https://bugs.launchpad.net/bugs/1115344. Approved by PS Jenkins bot, MC Return.	Sami Jaktholm	11 years ago
3721	Fade: Fix a typo that causes fade to constantly damage windo... Fade: Fix a typo that causes fade to constantly damage windows with a modified saturation. Fixes: https://bugs.launchpad.net/bugs/1159054. Approved by PS Jenkins bot, Stephen M. Webb, MC Return.	Sami Jaktholm	11 years ago
3720	Cube code cleanup: Cube code cleanup: Replaced 0xffff with GLfloat MaxUShortFloat = static_cast <GLfloat> (std::numeric_limits <unsigned short>::max ());. Use prefix instead of postfix de- and increments. Declaration and assignment of variables in one line. Declare variables shortly before you need them. Merged if condition checks. Added missing breaks (just a style issue). Added and removed brackets. Added and removed newlines. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3719	Commands, minor code cleanup: Commands, minor code cleanup: Declaration and assignment of variables in one line. Return ASAP. Added and removed newlines. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3718	ccp code cleanup: ccp code cleanup: Declare variables if you need them, do not declare them if you return. Declare CCSSettingValue *val; outside the foreach loop. Use prefix instead of postfix increments. Merged if condition checks. Declaration and assignment of variables in one line. Removed redundant brackets. Added and removed newlines. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3717	Text plugin code cleanup: Text plugin code cleanup: Do not declare variables you do not need if you return anyway. Use prefix instead of postfix increments. Merged if condition checks. Declaration and assignment of variables in one line. Added and removed newlines. Removed redundant brackets. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3716	Grid: Check that composite and opengl ABIs are COMPIZ_COMPOS... Grid: Check that composite and opengl ABIs are COMPIZ_COMPOSITE_ABI and COMPIZ_OPENGL_ABI instead of CORE_ABIVERSION. Fixes: https://bugs.launchpad.net/bugs/1181461. Approved by PS Jenkins bot, Sam Spilsbury.	Sami Jaktholm	11 years ago
3715	Titleinfo, minor code cleanup: Titleinfo, minor code cleanup: Declaration and assignment of local variables in one line. Removed redundant brackets. Added and removed newlines. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3714	Annotate plugin, code cleanup: Annotate plugin, code cleanup: Use prefix instead of postfix increment. Declaration and assignment of local variables in one line. Removed redundant brackets. Added and removed newlines if appropriate. Fixed indentation. Annotate plugin, fixes: Use GL_LINE_LOOP to draw the rectangle outline as glRecti is not supported on GLES. Simplified the coordinate calculation as the offset is not really needed here. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3713	Mousepoll code cleanup: Mousepoll code cleanup: Removed the unused variables MP_OPTION_MOUSE_POLL_INTERVAL and MP_OPTION_NUM. Declaration and assignment of variables in one line. Added and removed newlines. Removed redundant brackets. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3712	gtk-window-decorator: destroy action menu when any of the (c... gtk-window-decorator: destroy action menu when any of the (close, min, max) buttons on the title bar is pressed. Fixes: https://bugs.launchpad.net/bugs/1101648. Approved by PS Jenkins bot, Sam Spilsbury.	Hu Kang	11 years ago
3711	EZoom plugin, code improvements: EZoom plugin, code improvements: Bail out of functions ASAP, do not calculate stuff you might not need. Replaced: if (zoom_level == 1.0f && zooms.at (out).newZoom == 1.0f) with: if (zoom_level == zooms.at (out).newZoom) because we always want to get out of here if the target level is the same as current zoom level, not only in the case both are 1.0f. EZoom plugin, code cleanup: Assignment and declaration of variables in one line, if possible. Added and removed some newlines. Merged if condition checks. Indentation fixes. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3710	Wallpaper .xml.in cleanup: Wallpaper .xml.in cleanup: Better plugin description. Improved option names. Better wording of tooltips. Removed redundant newlines. Wallpaper code cleanup: Use prefix instead of postfix increments. Declaration and assignment of variables in one line. Added some newlines to improve readability. Removed redundant brackets. Fixed indentation. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return	11 years ago
3709	Add a small compiz_autopilot_acceptance_tests binary based o... Add a small compiz_autopilot_acceptance_tests binary based on gtest. This will find autopilot and run a predefined list of tests, wrapped up in the familiar google test format. The predefined list is basically a bunch of tests which effectively interact with the window manager. (LP: #1170013). Fixes: https://bugs.launchpad.net/bugs/1170013. Approved by PS Jenkins bot, Łukasz Zemczak, MC Return.	Sam Spilsbury	11 years ago
3708	GLScreen: break the iteration as soon as we've found a valid... GLScreen: break the iteration as soon as we've found a valid egl config. Approved by PS Jenkins bot, Sam Spilsbury.	Marco Trevisan (Trev...	11 years ago

Older »