Evas engine drm: Fix style (and thus compilation warning).
authorTom Hacohen <tom@stosb.com>
Tue, 30 Aug 2016 12:39:37 +0000 (13:39 +0100)
committerTom Hacohen <tom@stosb.com>
Tue, 30 Aug 2016 12:39:39 +0000 (13:39 +0100)
Double parenthesis in ifs, such as "if ((x = 5))" should only be used
iff using assignment inside of conditions (like the example above). This
explicitly conveys the intention to both the compiler and other
programmers and essentially eliminates the class of bugs that result
from accidental assignment when a comparison was intended and vise-versa
in conditions.

src/modules/evas/engines/drm/evas_outbuf.c

index 3ab2a54..e26fea7 100644 (file)
@@ -445,7 +445,7 @@ _outbuf_update_region_new(Outbuf *ob, int x, int y, int w, int h, int *cx, int *
 
    RECTS_CLIP_TO_RECT(x, y, w, h, 0, 0, ob->w, ob->h);
 
-   if ((ob->rotation == 0))// && (ob->depth == 32))
+   if (ob->rotation == 0) // && (ob->depth == 32))
      {
         Eina_Rectangle *rect;