[mutter/wip/carlosg/clip-me-harder: 10/10] window-actor: Remove negation in function name
- From: Carlos Garnacho <carlosg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [mutter/wip/carlosg/clip-me-harder: 10/10] window-actor: Remove negation in function name
- Date: Tue, 30 Jul 2019 21:22:44 +0000 (UTC)
commit f501fdcc5646ab42fc0450efe6f82326c0170fac
Author: Carlos Garnacho <carlosg gnome org>
Date: Tue Jul 30 23:01:04 2019 +0200
window-actor: Remove negation in function name
Double negations are the spawn of the devil, and is_non_opaque() is
used like that to find out if it's opaque most often, change the
function name to see the glass half full.
https://gitlab.gnome.org/GNOME/mutter/merge_requests/698
src/compositor/meta-window-actor.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
---
diff --git a/src/compositor/meta-window-actor.c b/src/compositor/meta-window-actor.c
index 0dcfe3d83..d518fec7c 100644
--- a/src/compositor/meta-window-actor.c
+++ b/src/compositor/meta-window-actor.c
@@ -288,13 +288,13 @@ is_argb32 (MetaWindowActor *self)
}
static gboolean
-is_non_opaque (MetaWindowActor *self)
+is_opaque (MetaWindowActor *self)
{
MetaWindowActorPrivate *priv =
meta_window_actor_get_instance_private (self);
MetaWindow *window = priv->window;
- return is_argb32 (self) || (window->opacity != 0xFF);
+ return !is_argb32 (self) && (window->opacity == 0xFF);
}
static gboolean
@@ -650,7 +650,7 @@ clip_shadow_under_window (MetaWindowActor *self)
if (priv->window->frame)
return TRUE;
- return !is_non_opaque (self);
+ return is_opaque (self);
}
static void
@@ -791,7 +791,7 @@ meta_window_actor_has_shadow (MetaWindowActor *self)
* Do not add shadows to non-opaque (ARGB32) windows, as we can't easily
* generate shadows for them.
*/
- if (is_non_opaque (self))
+ if (!is_opaque (self))
return FALSE;
/*
@@ -1376,7 +1376,7 @@ meta_window_actor_cull_out (MetaCullable *cullable,
meta_cullable_cull_out_children (cullable, unobscured_region, clip_region);
meta_window_actor_set_clip_region_beneath (self, clip_region);
- if (unobscured_region && !is_non_opaque (self))
+ if (unobscured_region && is_opaque (self))
{
cairo_region_t *region = meta_window_get_frame_bounds (priv->window);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]