ClangFormat: apply to source, most of intern

Apply clang format as proposed in T53211.

For details on usage and instructions for migrating branches
without conflicts, see:

https://wiki.blender.org/wiki/Tools/ClangFormat

1 files changed, 450 insertions, 444 deletions

diff --git a/source/blender/editors/space_node/node_view.c b/source/blender/editors/space_node/node_view.c
index 9d4b9d18c2a..15eb12ecf39 100644
--- a/source/blender/editors/space_node/node_view.c
+++ b/source/blender/editors/space_node/node_view.c
@@ -33,7 +33,7 @@
 #include "BKE_node.h"
 #include "BKE_screen.h"
 
-#include "ED_node.h"  /* own include */
+#include "ED_node.h" /* own include */
 #include "ED_screen.h"
 #include "ED_space_api.h"
 #include "ED_image.h"
@@ -52,587 +52,593 @@
 #include "IMB_imbuf.h"
 #include "IMB_imbuf_types.h"
 
-#include "node_intern.h"  /* own include */
+#include "node_intern.h" /* own include */
 #include "NOD_composite.h"
 
-
 /* **************** View All Operator ************** */
 
-int space_node_view_flag(bContext *C, SpaceNode *snode, ARegion *ar,
-                         const int node_flag, const int smooth_viewtx)
+int space_node_view_flag(
+    bContext *C, SpaceNode *snode, ARegion *ar, const int node_flag, const int smooth_viewtx)
 {
-	bNode *node;
-	rctf cur_new;
-	float oldwidth, oldheight, width, height;
-	float oldasp, asp;
-	int tot = 0;
-	bool has_frame = false;
-
-	oldwidth  = BLI_rctf_size_x(&ar->v2d.cur);
-	oldheight = BLI_rctf_size_y(&ar->v2d.cur);
-
-	oldasp = oldwidth / oldheight;
-
-	BLI_rctf_init_minmax(&cur_new);
-
-	if (snode->edittree) {
-		for (node = snode->edittree->nodes.first; node; node = node->next) {
-			if ((node->flag & node_flag) == node_flag) {
-				BLI_rctf_union(&cur_new, &node->totr);
-				tot++;
-
-				if (node->type == NODE_FRAME) {
-					has_frame = true;
-				}
-			}
-		}
-	}
-
-	if (tot) {
-		width  = BLI_rctf_size_x(&cur_new);
-		height = BLI_rctf_size_y(&cur_new);
-		asp = width / height;
-
-		/* for single non-frame nodes, don't zoom in, just pan view,
-		 * but do allow zooming out, this allows for big nodes to be zoomed out */
-		if ((tot == 1) &&
-		    (has_frame == false) &&
-		    ((oldwidth * oldheight) > (width * height)))
-		{
-			/* center, don't zoom */
-			BLI_rctf_resize(&cur_new, oldwidth, oldheight);
-		}
-		else {
-			if (oldasp < asp) {
-				const float height_new = width / oldasp;
-				cur_new.ymin = cur_new.ymin - height_new / 2.0f;
-				cur_new.ymax = cur_new.ymax + height_new / 2.0f;
-			}
-			else {
-				const float width_new = height * oldasp;
-				cur_new.xmin = cur_new.xmin - width_new / 2.0f;
-				cur_new.xmax = cur_new.xmax + width_new / 2.0f;
-			}
-
-			/* add some padding */
-			BLI_rctf_scale(&cur_new, 1.1f);
-		}
-
-		UI_view2d_smooth_view(C, ar, &cur_new, smooth_viewtx);
-	}
-
-	return (tot != 0);
+  bNode *node;
+  rctf cur_new;
+  float oldwidth, oldheight, width, height;
+  float oldasp, asp;
+  int tot = 0;
+  bool has_frame = false;
+
+  oldwidth = BLI_rctf_size_x(&ar->v2d.cur);
+  oldheight = BLI_rctf_size_y(&ar->v2d.cur);
+
+  oldasp = oldwidth / oldheight;
+
+  BLI_rctf_init_minmax(&cur_new);
+
+  if (snode->edittree) {
+    for (node = snode->edittree->nodes.first; node; node = node->next) {
+      if ((node->flag & node_flag) == node_flag) {
+        BLI_rctf_union(&cur_new, &node->totr);
+        tot++;
+
+        if (node->type == NODE_FRAME) {
+          has_frame = true;
+        }
+      }
+    }
+  }
+
+  if (tot) {
+    width = BLI_rctf_size_x(&cur_new);
+    height = BLI_rctf_size_y(&cur_new);
+    asp = width / height;
+
+    /* for single non-frame nodes, don't zoom in, just pan view,
+     * but do allow zooming out, this allows for big nodes to be zoomed out */
+    if ((tot == 1) && (has_frame == false) && ((oldwidth * oldheight) > (width * height))) {
+      /* center, don't zoom */
+      BLI_rctf_resize(&cur_new, oldwidth, oldheight);
+    }
+    else {
+      if (oldasp < asp) {
+        const float height_new = width / oldasp;
+        cur_new.ymin = cur_new.ymin - height_new / 2.0f;
+        cur_new.ymax = cur_new.ymax + height_new / 2.0f;
+      }
+      else {
+        const float width_new = height * oldasp;
+        cur_new.xmin = cur_new.xmin - width_new / 2.0f;
+        cur_new.xmax = cur_new.xmax + width_new / 2.0f;
+      }
+
+      /* add some padding */
+      BLI_rctf_scale(&cur_new, 1.1f);
+    }
+
+    UI_view2d_smooth_view(C, ar, &cur_new, smooth_viewtx);
+  }
+
+  return (tot != 0);
 }
 
 static int node_view_all_exec(bContext *C, wmOperator *op)
 {
-	ARegion *ar = CTX_wm_region(C);
-	SpaceNode *snode = CTX_wm_space_node(C);
-	const int smooth_viewtx = WM_operator_smooth_viewtx_get(op);
-
-	/* is this really needed? */
-	snode->xof = 0;
-	snode->yof = 0;
-
-	if (space_node_view_flag(C, snode, ar, 0, smooth_viewtx)) {
-		return OPERATOR_FINISHED;
-	}
-	else {
-		return OPERATOR_CANCELLED;
-	}
+  ARegion *ar = CTX_wm_region(C);
+  SpaceNode *snode = CTX_wm_space_node(C);
+  const int smooth_viewtx = WM_operator_smooth_viewtx_get(op);
+
+  /* is this really needed? */
+  snode->xof = 0;
+  snode->yof = 0;
+
+  if (space_node_view_flag(C, snode, ar, 0, smooth_viewtx)) {
+    return OPERATOR_FINISHED;
+  }
+  else {
+    return OPERATOR_CANCELLED;
+  }
 }
 
 void NODE_OT_view_all(wmOperatorType *ot)
 {
-	/* identifiers */
-	ot->name = "View All";
-	ot->idname = "NODE_OT_view_all";
-	ot->description = "Resize view so you can see all nodes";
+  /* identifiers */
+  ot->name = "View All";
+  ot->idname = "NODE_OT_view_all";
+  ot->description = "Resize view so you can see all nodes";
 
-	/* api callbacks */
-	ot->exec = node_view_all_exec;
-	ot->poll = ED_operator_node_active;
+  /* api callbacks */
+  ot->exec = node_view_all_exec;
+  ot->poll = ED_operator_node_active;
 
-	/* flags */
-	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
+  /* flags */
+  ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 }
 
 static int node_view_selected_exec(bContext *C, wmOperator *op)
 {
-	ARegion *ar = CTX_wm_region(C);
-	SpaceNode *snode = CTX_wm_space_node(C);
-	const int smooth_viewtx = WM_operator_smooth_viewtx_get(op);
-
-	if (space_node_view_flag(C, snode, ar, NODE_SELECT, smooth_viewtx)) {
-		return OPERATOR_FINISHED;
-	}
-	else {
-		return OPERATOR_CANCELLED;
-	}
+  ARegion *ar = CTX_wm_region(C);
+  SpaceNode *snode = CTX_wm_space_node(C);
+  const int smooth_viewtx = WM_operator_smooth_viewtx_get(op);
+
+  if (space_node_view_flag(C, snode, ar, NODE_SELECT, smooth_viewtx)) {
+    return OPERATOR_FINISHED;
+  }
+  else {
+    return OPERATOR_CANCELLED;
+  }
 }
 
 void NODE_OT_view_selected(wmOperatorType *ot)
 {
-	/* identifiers */
-	ot->name = "View Selected";
-	ot->idname = "NODE_OT_view_selected";
-	ot->description = "Resize view so you can see selected nodes";
+  /* identifiers */
+  ot->name = "View Selected";
+  ot->idname = "NODE_OT_view_selected";
+  ot->description = "Resize view so you can see selected nodes";
 
-	/* api callbacks */
-	ot->exec = node_view_selected_exec;
-	ot->poll = ED_operator_node_active;
+  /* api callbacks */
+  ot->exec = node_view_selected_exec;
+  ot->poll = ED_operator_node_active;
 
-	/* flags */
-	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
+  /* flags */
+  ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 }
 
 /* **************** Background Image Operators ************** */
 
 typedef struct NodeViewMove {
-	int mvalo[2];
-	int xmin, ymin, xmax, ymax;
+  int mvalo[2];
+  int xmin, ymin, xmax, ymax;
 } NodeViewMove;
 
 static int snode_bg_viewmove_modal(bContext *C, wmOperator *op, const wmEvent *event)
 {
-	SpaceNode *snode = CTX_wm_space_node(C);
-	ARegion *ar = CTX_wm_region(C);
-	NodeViewMove *nvm = op->customdata;
-
-	switch (event->type) {
-		case MOUSEMOVE:
-
-			snode->xof -= (nvm->mvalo[0] - event->mval[0]);
-			snode->yof -= (nvm->mvalo[1] - event->mval[1]);
-			nvm->mvalo[0] = event->mval[0];
-			nvm->mvalo[1] = event->mval[1];
-
-			/* prevent dragging image outside of the window and losing it! */
-			CLAMP(snode->xof, nvm->xmin, nvm->xmax);
-			CLAMP(snode->yof, nvm->ymin, nvm->ymax);
-
-			ED_region_tag_redraw(ar);
-			WM_main_add_notifier(NC_NODE | ND_DISPLAY, NULL);
-
-			break;
-
-		case LEFTMOUSE:
-		case MIDDLEMOUSE:
-		case RIGHTMOUSE:
-			if (event->val == KM_RELEASE) {
-				MEM_freeN(nvm);
-				op->customdata = NULL;
-				return OPERATOR_FINISHED;
-			}
-			break;
-	}
-
-	return OPERATOR_RUNNING_MODAL;
+  SpaceNode *snode = CTX_wm_space_node(C);
+  ARegion *ar = CTX_wm_region(C);
+  NodeViewMove *nvm = op->customdata;
+
+  switch (event->type) {
+    case MOUSEMOVE:
+
+      snode->xof -= (nvm->mvalo[0] - event->mval[0]);
+      snode->yof -= (nvm->mvalo[1] - event->mval[1]);
+      nvm->mvalo[0] = event->mval[0];
+      nvm->mvalo[1] = event->mval[1];
+
+      /* prevent dragging image outside of the window and losing it! */
+      CLAMP(snode->xof, nvm->xmin, nvm->xmax);
+      CLAMP(snode->yof, nvm->ymin, nvm->ymax);
+
+      ED_region_tag_redraw(ar);
+      WM_main_add_notifier(NC_NODE | ND_DISPLAY, NULL);
+
+      break;
+
+    case LEFTMOUSE:
+    case MIDDLEMOUSE:
+    case RIGHTMOUSE:
+      if (event->val == KM_RELEASE) {
+        MEM_freeN(nvm);
+        op->customdata = NULL;
+        return OPERATOR_FINISHED;
+      }
+      break;
+  }
+
+  return OPERATOR_RUNNING_MODAL;
 }
 
 static int snode_bg_viewmove_invoke(bContext *C, wmOperator *op, const wmEvent *event)
 {
-	Main *bmain = CTX_data_main(C);
-	SpaceNode *snode = CTX_wm_space_node(C);
-	ARegion *ar = CTX_wm_region(C);
-	NodeViewMove *nvm;
-	Image *ima;
-	ImBuf *ibuf;
-	const float pad = 32.0f; /* better be bigger then scrollbars */
+  Main *bmain = CTX_data_main(C);
+  SpaceNode *snode = CTX_wm_space_node(C);
+  ARegion *ar = CTX_wm_region(C);
+  NodeViewMove *nvm;
+  Image *ima;
+  ImBuf *ibuf;
+  const float pad = 32.0f; /* better be bigger then scrollbars */
 
-	void *lock;
+  void *lock;
 
-	ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
-	ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
+  ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
+  ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
 
-	if (ibuf == NULL) {
-		BKE_image_release_ibuf(ima, ibuf, lock);
-		return OPERATOR_CANCELLED;
-	}
+  if (ibuf == NULL) {
+    BKE_image_release_ibuf(ima, ibuf, lock);
+    return OPERATOR_CANCELLED;
+  }
 
-	nvm = MEM_callocN(sizeof(NodeViewMove), "NodeViewMove struct");
-	op->customdata = nvm;
-	nvm->mvalo[0] = event->mval[0];
-	nvm->mvalo[1] = event->mval[1];
+  nvm = MEM_callocN(sizeof(NodeViewMove), "NodeViewMove struct");
+  op->customdata = nvm;
+  nvm->mvalo[0] = event->mval[0];
+  nvm->mvalo[1] = event->mval[1];
 
-	nvm->xmin = -(ar->winx / 2) - (ibuf->x * (0.5f * snode->zoom)) + pad;
-	nvm->xmax =  (ar->winx / 2) + (ibuf->x * (0.5f * snode->zoom)) - pad;
-	nvm->ymin = -(ar->winy / 2) - (ibuf->y * (0.5f * snode->zoom)) + pad;
-	nvm->ymax =  (ar->winy / 2) + (ibuf->y * (0.5f * snode->zoom)) - pad;
+  nvm->xmin = -(ar->winx / 2) - (ibuf->x * (0.5f * snode->zoom)) + pad;
+  nvm->xmax = (ar->winx / 2) + (ibuf->x * (0.5f * snode->zoom)) - pad;
+  nvm->ymin = -(ar->winy / 2) - (ibuf->y * (0.5f * snode->zoom)) + pad;
+  nvm->ymax = (ar->winy / 2) + (ibuf->y * (0.5f * snode->zoom)) - pad;
 
-	BKE_image_release_ibuf(ima, ibuf, lock);
+  BKE_image_release_ibuf(ima, ibuf, lock);
 
-	/* add modal handler */
-	WM_event_add_modal_handler(C, op);
+  /* add modal handler */
+  WM_event_add_modal_handler(C, op);
 
-	return OPERATOR_RUNNING_MODAL;
+  return OPERATOR_RUNNING_MODAL;
 }
 
 static void snode_bg_viewmove_cancel(bContext *UNUSED(C), wmOperator *op)
 {
-	MEM_freeN(op->customdata);
-	op->customdata = NULL;
+  MEM_freeN(op->customdata);
+  op->customdata = NULL;
 }
 
 void NODE_OT_backimage_move(wmOperatorType *ot)
 {
-	/* identifiers */
-	ot->name = "Background Image Move";
-	ot->description = "Move Node backdrop";
-	ot->idname = "NODE_OT_backimage_move";
-
-	/* api callbacks */
-	ot->invoke = snode_bg_viewmove_invoke;
-	ot->modal = snode_bg_viewmove_modal;
-	ot->poll = composite_node_active;
-	ot->cancel = snode_bg_viewmove_cancel;
-
-	/* flags */
-	ot->flag = OPTYPE_BLOCKING | OPTYPE_GRAB_CURSOR;
+  /* identifiers */
+  ot->name = "Background Image Move";
+  ot->description = "Move Node backdrop";
+  ot->idname = "NODE_OT_backimage_move";
+
+  /* api callbacks */
+  ot->invoke = snode_bg_viewmove_invoke;
+  ot->modal = snode_bg_viewmove_modal;
+  ot->poll = composite_node_active;
+  ot->cancel = snode_bg_viewmove_cancel;
+
+  /* flags */
+  ot->flag = OPTYPE_BLOCKING | OPTYPE_GRAB_CURSOR;
 }
 
 static int backimage_zoom_exec(bContext *C, wmOperator *op)
 {
-	SpaceNode *snode = CTX_wm_space_node(C);
-	ARegion *ar = CTX_wm_region(C);
-	float fac = RNA_float_get(op->ptr, "factor");
+  SpaceNode *snode = CTX_wm_space_node(C);
+  ARegion *ar = CTX_wm_region(C);
+  float fac = RNA_float_get(op->ptr, "factor");
 
-	snode->zoom *= fac;
-	ED_region_tag_redraw(ar);
-	WM_main_add_notifier(NC_NODE | ND_DISPLAY, NULL);
+  snode->zoom *= fac;
+  ED_region_tag_redraw(ar);
+  WM_main_add_notifier(NC_NODE | ND_DISPLAY, NULL);
 
-	return OPERATOR_FINISHED;
+  return OPERATOR_FINISHED;
 }
 
-
 void NODE_OT_backimage_zoom(wmOperatorType *ot)
 {
 
-	/* identifiers */
-	ot->name = "Background Image Zoom";
-	ot->idname = "NODE_OT_backimage_zoom";
-	ot->description = "Zoom in/out the background image";
+  /* identifiers */
+  ot->name = "Background Image Zoom";
+  ot->idname = "NODE_OT_backimage_zoom";
+  ot->description = "Zoom in/out the background image";
 
-	/* api callbacks */
-	ot->exec = backimage_zoom_exec;
-	ot->poll = composite_node_active;
+  /* api callbacks */
+  ot->exec = backimage_zoom_exec;
+  ot->poll = composite_node_active;
 
-	/* flags */
-	ot->flag = OPTYPE_BLOCKING;
+  /* flags */
+  ot->flag = OPTYPE_BLOCKING;
 
-	/* internal */
-	RNA_def_float(ot->srna, "factor", 1.2f, 0.0f, 10.0f, "Factor", "", 0.0f, 10.0f);
+  /* internal */
+  RNA_def_float(ot->srna, "factor", 1.2f, 0.0f, 10.0f, "Factor", "", 0.0f, 10.0f);
 }
 
 static int backimage_fit_exec(bContext *C, wmOperator *UNUSED(op))
 {
-	Main *bmain = CTX_data_main(C);
-	SpaceNode *snode = CTX_wm_space_node(C);
-	ARegion *ar = CTX_wm_region(C);
+  Main *bmain = CTX_data_main(C);
+  SpaceNode *snode = CTX_wm_space_node(C);
+  ARegion *ar = CTX_wm_region(C);
 
-	Image *ima;
-	ImBuf *ibuf;
+  Image *ima;
+  ImBuf *ibuf;
 
-	const float pad = 32.0f;
+  const float pad = 32.0f;
 
-	void *lock;
+  void *lock;
 
-	float facx, facy;
+  float facx, facy;
 
-	ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
-	ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
+  ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
+  ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
 
-	if ((ibuf == NULL) || (ibuf->x == 0) || (ibuf->y == 0)) {
-		BKE_image_release_ibuf(ima, ibuf, lock);
-		return OPERATOR_CANCELLED;
-	}
+  if ((ibuf == NULL) || (ibuf->x == 0) || (ibuf->y == 0)) {
+    BKE_image_release_ibuf(ima, ibuf, lock);
+    return OPERATOR_CANCELLED;
+  }
 
-	facx = 1.0f * (ar->sizex - pad) / (ibuf->x * snode->zoom);
-	facy = 1.0f * (ar->sizey - pad) / (ibuf->y * snode->zoom);
+  facx = 1.0f * (ar->sizex - pad) / (ibuf->x * snode->zoom);
+  facy = 1.0f * (ar->sizey - pad) / (ibuf->y * snode->zoom);
 
-	BKE_image_release_ibuf(ima, ibuf, lock);
+  BKE_image_release_ibuf(ima, ibuf, lock);
 
-	snode->zoom *= min_ff(facx, facy);
+  snode->zoom *= min_ff(facx, facy);
 
-	snode->xof = 0;
-	snode->yof = 0;
+  snode->xof = 0;
+  snode->yof = 0;
 
-	ED_region_tag_redraw(ar);
-	WM_main_add_notifier(NC_NODE | ND_DISPLAY, NULL);
+  ED_region_tag_redraw(ar);
+  WM_main_add_notifier(NC_NODE | ND_DISPLAY, NULL);
 
-	return OPERATOR_FINISHED;
+  return OPERATOR_FINISHED;
 }
 
 void NODE_OT_backimage_fit(wmOperatorType *ot)
 {
 
-	/* identifiers */
-	ot->name = "Background Image Fit";
-	ot->idname = "NODE_OT_backimage_fit";
-	ot->description = "Fit the background image to the view";
-
-	/* api callbacks */
-	ot->exec = backimage_fit_exec;
-	ot->poll = composite_node_active;
+  /* identifiers */
+  ot->name = "Background Image Fit";
+  ot->idname = "NODE_OT_backimage_fit";
+  ot->description = "Fit the background image to the view";
 
-	/* flags */
-	ot->flag = OPTYPE_BLOCKING;
+  /* api callbacks */
+  ot->exec = backimage_fit_exec;
+  ot->poll = composite_node_active;
 
+  /* flags */
+  ot->flag = OPTYPE_BLOCKING;
 }
 
 /******************** sample backdrop operator ********************/
 
 typedef struct ImageSampleInfo {
-	ARegionType *art;
-	void *draw_handle;
-	int x, y;
-	int channels;
+  ARegionType *art;
+  void *draw_handle;
+  int x, y;
+  int channels;
 
-	unsigned char col[4];
-	float colf[4];
-	float linearcol[4];
+  unsigned char col[4];
+  float colf[4];
+  float linearcol[4];
 
-	int z;
-	float zf;
+  int z;
+  float zf;
 
-	int *zp;
-	float *zfp;
+  int *zp;
+  float *zfp;
 
-	int draw;
-	int color_manage;
+  int draw;
+  int color_manage;
 } ImageSampleInfo;
 
 static void sample_draw(const bContext *C, ARegion *ar, void *arg_info)
 {
-	Scene *scene = CTX_data_scene(C);
-	ImageSampleInfo *info = arg_info;
-
-	if (info->draw) {
-		ED_image_draw_info(scene, ar, info->color_manage, false, info->channels,
-		                   info->x, info->y, info->col, info->colf, info->linearcol,
-		                   info->zp, info->zfp);
-	}
+  Scene *scene = CTX_data_scene(C);
+  ImageSampleInfo *info = arg_info;
+
+  if (info->draw) {
+    ED_image_draw_info(scene,
+                       ar,
+                       info->color_manage,
+                       false,
+                       info->channels,
+                       info->x,
+                       info->y,
+                       info->col,
+                       info->colf,
+                       info->linearcol,
+                       info->zp,
+                       info->zfp);
+  }
 }
 
 /* Returns color in linear space, matching ED_space_image_color_sample().
  * And here we've got recursion in the comments tips...
  */
-bool ED_space_node_color_sample(Main *bmain, SpaceNode *snode, ARegion *ar, int mval[2], float r_col[3])
+bool ED_space_node_color_sample(
+    Main *bmain, SpaceNode *snode, ARegion *ar, int mval[2], float r_col[3])
 {
-	void *lock;
-	Image *ima;
-	ImBuf *ibuf;
-	float fx, fy, bufx, bufy;
-	bool ret = false;
-
-	if (!ED_node_is_compositor(snode) || (snode->flag & SNODE_BACKDRAW) == 0) {
-		/* use viewer image for color sampling only if we're in compositor tree
-		 * with backdrop enabled
-		 */
-		return false;
-	}
-
-	ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
-	ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
-	if (!ibuf) {
-		return false;
-	}
-
-	/* map the mouse coords to the backdrop image space */
-	bufx = ibuf->x * snode->zoom;
-	bufy = ibuf->y * snode->zoom;
-	fx = (bufx > 0.0f ? ((float)mval[0] - 0.5f * ar->winx - snode->xof) / bufx + 0.5f : 0.0f);
-	fy = (bufy > 0.0f ? ((float)mval[1] - 0.5f * ar->winy - snode->yof) / bufy + 0.5f : 0.0f);
-
-	if (fx >= 0.0f && fy >= 0.0f && fx < 1.0f && fy < 1.0f) {
-		const float *fp;
-		unsigned char *cp;
-		int x = (int)(fx * ibuf->x), y = (int)(fy * ibuf->y);
-
-		CLAMP(x, 0, ibuf->x - 1);
-		CLAMP(y, 0, ibuf->y - 1);
-
-		if (ibuf->rect_float) {
-			fp = (ibuf->rect_float + (ibuf->channels) * (y * ibuf->x + x));
-			/* IB_PROFILE_NONE is default but infact its linear */
-			copy_v3_v3(r_col, fp);
-			ret = true;
-		}
-		else if (ibuf->rect) {
-			cp = (unsigned char *)(ibuf->rect + y * ibuf->x + x);
-			rgb_uchar_to_float(r_col, cp);
-			IMB_colormanagement_colorspace_to_scene_linear_v3(r_col, ibuf->rect_colorspace);
-			ret = true;
-		}
-	}
-
-	BKE_image_release_ibuf(ima, ibuf, lock);
-
-	return ret;
+  void *lock;
+  Image *ima;
+  ImBuf *ibuf;
+  float fx, fy, bufx, bufy;
+  bool ret = false;
+
+  if (!ED_node_is_compositor(snode) || (snode->flag & SNODE_BACKDRAW) == 0) {
+    /* use viewer image for color sampling only if we're in compositor tree
+     * with backdrop enabled
+     */
+    return false;
+  }
+
+  ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
+  ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
+  if (!ibuf) {
+    return false;
+  }
+
+  /* map the mouse coords to the backdrop image space */
+  bufx = ibuf->x * snode->zoom;
+  bufy = ibuf->y * snode->zoom;
+  fx = (bufx > 0.0f ? ((float)mval[0] - 0.5f * ar->winx - snode->xof) / bufx + 0.5f : 0.0f);
+  fy = (bufy > 0.0f ? ((float)mval[1] - 0.5f * ar->winy - snode->yof) / bufy + 0.5f : 0.0f);
+
+  if (fx >= 0.0f && fy >= 0.0f && fx < 1.0f && fy < 1.0f) {
+    const float *fp;
+    unsigned char *cp;
+    int x = (int)(fx * ibuf->x), y = (int)(fy * ibuf->y);
+
+    CLAMP(x, 0, ibuf->x - 1);
+    CLAMP(y, 0, ibuf->y - 1);
+
+    if (ibuf->rect_float) {
+      fp = (ibuf->rect_float + (ibuf->channels) * (y * ibuf->x + x));
+      /* IB_PROFILE_NONE is default but infact its linear */
+      copy_v3_v3(r_col, fp);
+      ret = true;
+    }
+    else if (ibuf->rect) {
+      cp = (unsigned char *)(ibuf->rect + y * ibuf->x + x);
+      rgb_uchar_to_float(r_col, cp);
+      IMB_colormanagement_colorspace_to_scene_linear_v3(r_col, ibuf->rect_colorspace);
+      ret = true;
+    }
+  }
+
+  BKE_image_release_ibuf(ima, ibuf, lock);
+
+  return ret;
 }
 
 static void sample_apply(bContext *C, wmOperator *op, const wmEvent *event)
 {
-	Main *bmain = CTX_data_main(C);
-	SpaceNode *snode = CTX_wm_space_node(C);
-	ARegion *ar = CTX_wm_region(C);
-	ImageSampleInfo *info = op->customdata;
-	void *lock;
-	Image *ima;
-	ImBuf *ibuf;
-	float fx, fy, bufx, bufy;
-
-	ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
-	ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
-	if (!ibuf) {
-		info->draw = 0;
-		return;
-	}
-
-	if (!ibuf->rect) {
-		IMB_rect_from_float(ibuf);
-	}
-
-	/* map the mouse coords to the backdrop image space */
-	bufx = ibuf->x * snode->zoom;
-	bufy = ibuf->y * snode->zoom;
-	fx = (bufx > 0.0f ? ((float)event->mval[0] - 0.5f * ar->winx - snode->xof) / bufx + 0.5f : 0.0f);
-	fy = (bufy > 0.0f ? ((float)event->mval[1] - 0.5f * ar->winy - snode->yof) / bufy + 0.5f : 0.0f);
-
-	if (fx >= 0.0f && fy >= 0.0f && fx < 1.0f && fy < 1.0f) {
-		const float *fp;
-		unsigned char *cp;
-		int x = (int)(fx * ibuf->x), y = (int)(fy * ibuf->y);
-
-		CLAMP(x, 0, ibuf->x - 1);
-		CLAMP(y, 0, ibuf->y - 1);
-
-		info->x = x;
-		info->y = y;
-		info->draw = 1;
-		info->channels = ibuf->channels;
-
-		info->zp = NULL;
-		info->zfp = NULL;
-
-		if (ibuf->rect) {
-			cp = (unsigned char *)(ibuf->rect + y * ibuf->x + x);
-
-			info->col[0] = cp[0];
-			info->col[1] = cp[1];
-			info->col[2] = cp[2];
-			info->col[3] = cp[3];
-
-			info->colf[0] = (float)cp[0] / 255.0f;
-			info->colf[1] = (float)cp[1] / 255.0f;
-			info->colf[2] = (float)cp[2] / 255.0f;
-			info->colf[3] = (float)cp[3] / 255.0f;
-
-			copy_v4_v4(info->linearcol, info->colf);
-			IMB_colormanagement_colorspace_to_scene_linear_v4(info->linearcol, false, ibuf->rect_colorspace);
-
-			info->color_manage = true;
-		}
-		if (ibuf->rect_float) {
-			fp = (ibuf->rect_float + (ibuf->channels) * (y * ibuf->x + x));
-
-			info->colf[0] = fp[0];
-			info->colf[1] = fp[1];
-			info->colf[2] = fp[2];
-			info->colf[3] = fp[3];
-
-			info->color_manage = true;
-		}
-
-		if (ibuf->zbuf) {
-			info->z = ibuf->zbuf[y * ibuf->x + x];
-			info->zp = &info->z;
-		}
-		if (ibuf->zbuf_float) {
-			info->zf = ibuf->zbuf_float[y * ibuf->x + x];
-			info->zfp = &info->zf;
-		}
-
-		ED_node_sample_set(info->colf);
-	}
-	else {
-		info->draw = 0;
-		ED_node_sample_set(NULL);
-	}
-
-	BKE_image_release_ibuf(ima, ibuf, lock);
-
-	ED_area_tag_redraw(CTX_wm_area(C));
+  Main *bmain = CTX_data_main(C);
+  SpaceNode *snode = CTX_wm_space_node(C);
+  ARegion *ar = CTX_wm_region(C);
+  ImageSampleInfo *info = op->customdata;
+  void *lock;
+  Image *ima;
+  ImBuf *ibuf;
+  float fx, fy, bufx, bufy;
+
+  ima = BKE_image_verify_viewer(bmain, IMA_TYPE_COMPOSITE, "Viewer Node");
+  ibuf = BKE_image_acquire_ibuf(ima, NULL, &lock);
+  if (!ibuf) {
+    info->draw = 0;
+    return;
+  }
+
+  if (!ibuf->rect) {
+    IMB_rect_from_float(ibuf);
+  }
+
+  /* map the mouse coords to the backdrop image space */
+  bufx = ibuf->x * snode->zoom;
+  bufy = ibuf->y * snode->zoom;
+  fx = (bufx > 0.0f ? ((float)event->mval[0] - 0.5f * ar->winx - snode->xof) / bufx + 0.5f : 0.0f);
+  fy = (bufy > 0.0f ? ((float)event->mval[1] - 0.5f * ar->winy - snode->yof) / bufy + 0.5f : 0.0f);
+
+  if (fx >= 0.0f && fy >= 0.0f && fx < 1.0f && fy < 1.0f) {
+    const float *fp;
+    unsigned char *cp;
+    int x = (int)(fx * ibuf->x), y = (int)(fy * ibuf->y);
+
+    CLAMP(x, 0, ibuf->x - 1);
+    CLAMP(y, 0, ibuf->y - 1);
+
+    info->x = x;
+    info->y = y;
+    info->draw = 1;
+    info->channels = ibuf->channels;
+
+    info->zp = NULL;
+    info->zfp = NULL;
+
+    if (ibuf->rect) {
+      cp = (unsigned char *)(ibuf->rect + y * ibuf->x + x);
+
+      info->col[0] = cp[0];
+      info->col[1] = cp[1];
+      info->col[2] = cp[2];
+      info->col[3] = cp[3];
+
+      info->colf[0] = (float)cp[0] / 255.0f;
+      info->colf[1] = (float)cp[1] / 255.0f;
+      info->colf[2] = (float)cp[2] / 255.0f;
+      info->colf[3] = (float)cp[3] / 255.0f;
+
+      copy_v4_v4(info->linearcol, info->colf);
+      IMB_colormanagement_colorspace_to_scene_linear_v4(
+          info->linearcol, false, ibuf->rect_colorspace);
+
+      info->color_manage = true;
+    }
+    if (ibuf->rect_float) {
+      fp = (ibuf->rect_float + (ibuf->channels) * (y * ibuf->x + x));
+
+      info->colf[0] = fp[0];
+      info->colf[1] = fp[1];
+      info->colf[2] = fp[2];
+      info->colf[3] = fp[3];
+
+      info->color_manage = true;
+    }
+
+    if (ibuf->zbuf) {
+      info->z = ibuf->zbuf[y * ibuf->x + x];
+      info->zp = &info->z;
+    }
+    if (ibuf->zbuf_float) {
+      info->zf = ibuf->zbuf_float[y * ibuf->x + x];
+      info->zfp = &info->zf;
+    }
+
+    ED_node_sample_set(info->colf);
+  }
+  else {
+    info->draw = 0;
+    ED_node_sample_set(NULL);
+  }
+
+  BKE_image_release_ibuf(ima, ibuf, lock);
+
+  ED_area_tag_redraw(CTX_wm_area(C));
 }
 
 static void sample_exit(bContext *C, wmOperator *op)
 {
-	ImageSampleInfo *info = op->customdata;
+  ImageSampleInfo *info = op->customdata;
 
-	ED_node_sample_set(NULL);
-	ED_region_draw_cb_exit(info->art, info->draw_handle);
-	ED_area_tag_redraw(CTX_wm_area(C));
-	MEM_freeN(info);
+  ED_node_sample_set(NULL);
+  ED_region_draw_cb_exit(info->art, info->draw_handle);
+  ED_area_tag_redraw(CTX_wm_area(C));
+  MEM_freeN(info);
 }
 
 static int sample_invoke(bContext *C, wmOperator *op, const wmEvent *event)
 {
-	SpaceNode *snode = CTX_wm_space_node(C);
-	ARegion *ar = CTX_wm_region(C);
-	ImageSampleInfo *info;
+  SpaceNode *snode = CTX_wm_space_node(C);
+  ARegion *ar = CTX_wm_region(C);
+  ImageSampleInfo *info;
 
-	if (!ED_node_is_compositor(snode) || !(snode->flag & SNODE_BACKDRAW)) {
-		return OPERATOR_CANCELLED;
-	}
+  if (!ED_node_is_compositor(snode) || !(snode->flag & SNODE_BACKDRAW)) {
+    return OPERATOR_CANCELLED;
+  }
 
-	info = MEM_callocN(sizeof(ImageSampleInfo), "ImageSampleInfo");
-	info->art = ar->type;
-	info->draw_handle = ED_region_draw_cb_activate(ar->type, sample_draw, info, REGION_DRAW_POST_PIXEL);
-	op->customdata = info;
+  info = MEM_callocN(sizeof(ImageSampleInfo), "ImageSampleInfo");
+  info->art = ar->type;
+  info->draw_handle = ED_region_draw_cb_activate(
+      ar->type, sample_draw, info, REGION_DRAW_POST_PIXEL);
+  op->customdata = info;
 
-	sample_apply(C, op, event);
+  sample_apply(C, op, event);
 
-	WM_event_add_modal_handler(C, op);
+  WM_event_add_modal_handler(C, op);
 
-	return OPERATOR_RUNNING_MODAL;
+  return OPERATOR_RUNNING_MODAL;
 }
 
 static int sample_modal(bContext *C, wmOperator *op, const wmEvent *event)
 {
-	switch (event->type) {
-		case LEFTMOUSE:
-		case RIGHTMOUSE: // XXX hardcoded
-			if (event->val == KM_RELEASE) {
-				sample_exit(C, op);
-				return OPERATOR_CANCELLED;
-			}
-			break;
-		case MOUSEMOVE:
-			sample_apply(C, op, event);
-			break;
-	}
-
-	return OPERATOR_RUNNING_MODAL;
+  switch (event->type) {
+    case LEFTMOUSE:
+    case RIGHTMOUSE:  // XXX hardcoded
+      if (event->val == KM_RELEASE) {
+        sample_exit(C, op);
+        return OPERATOR_CANCELLED;
+      }
+      break;
+    case MOUSEMOVE:
+      sample_apply(C, op, event);
+      break;
+  }
+
+  return OPERATOR_RUNNING_MODAL;
 }
 
 static void sample_cancel(bContext *C, wmOperator *op)
 {
-	sample_exit(C, op);
+  sample_exit(C, op);
 }
 
 void NODE_OT_backimage_sample(wmOperatorType *ot)
 {
-	/* identifiers */
-	ot->name = "Backimage Sample";
-	ot->idname = "NODE_OT_backimage_sample";
-	ot->description = "Use mouse to sample background image";
-
-	/* api callbacks */
-	ot->invoke = sample_invoke;
-	ot->modal = sample_modal;
-	ot->cancel = sample_cancel;
-	ot->poll = ED_operator_node_active;
-
-	/* flags */
-	ot->flag = OPTYPE_BLOCKING;
+  /* identifiers */
+  ot->name = "Backimage Sample";
+  ot->idname = "NODE_OT_backimage_sample";
+  ot->description = "Use mouse to sample background image";
+
+  /* api callbacks */
+  ot->invoke = sample_invoke;
+  ot->modal = sample_modal;
+  ot->cancel = sample_cancel;
+  ot->poll = ED_operator_node_active;
+
+  /* flags */
+  ot->flag = OPTYPE_BLOCKING;
 }