Differential D10548 Diff 34464 source/blender/blenlib/intern/array_utils.c

Changeset View

Standalone View

source/blender/blenlib/intern/array_utils.c

Show All 24 Lines
#include <stdlib.h>		#include <stdlib.h>
#include <string.h>		#include <string.h>

#include "MEM_guardedalloc.h"		#include "MEM_guardedalloc.h"

#include "BLI_array_utils.h"		#include "BLI_array_utils.h"

#include "BLI_alloca.h"		#include "BLI_alloca.h"
		#include "BLI_math_base.h"
#include "BLI_sys_types.h"		#include "BLI_sys_types.h"
#include "BLI_utildefines.h"		#include "BLI_utildefines.h"

#include "BLI_strict_flags.h"		#include "BLI_strict_flags.h"

/**		/**
*In-place array reverse.		*In-place array reverse.
*		*
▲ Show 20 Lines • Show All 272 Lines • ▼ Show 20 Lines	bool _bli_array_is_zeroed(const void *arr_v, unsigned int arr_len, size_t arr_stride)
size_t i = arr_stride * arr_len;		size_t i = arr_stride * arr_len;
while (i--) {		while (i--) {
if (*(arr_step++)) {		if (*(arr_step++)) {
return false;		return false;
}		}
}		}
return true;		return true;
}		}

		/**
		* Smart function to sample a rect spiraling outside, nice for selection ID.
		*
		* \param arr_shape: size [h, w].
		* \param center: coordinates [y, x] indicating where to start reading.
		brechtUnsubmitted Done Inline Actions Why flip the x/y order? I'm having a hard time reading the code since I have to mentally flip the axes all the time. brecht: Why flip the x/y order? I'm having a hard time reading the code since I have to mentally flip…
		mano-wiiAuthorUnsubmitted Done Inline Actions In fact, using the x, y order can be useful when, for example, the mouse coordinates are passed. But for most cases, the value of a buffer is obtained as follows: value = buff[h][w] Having the `h` before the `w`. If we consider `h` to be a height that is vertical and `w` to be a width (horizontal), the expected order to read a value from a buffer is `buff[y][x]`. But this is not mandatory. In fact, I had set the order x, y before. mano-wii: In fact, using the x, y order can be useful when, for example, the mouse coordinates are passed.
		brechtUnsubmitted Done Inline Actions Please use the standard x,y and w,h order we use everywhere else in Blender. brecht: Please use the standard x,y and w,h order we use everywhere else in Blender.
		*/
		bool _bli_array_iter_spiral_square(const void *arr_v,
		const int arr_shape[2],
		const size_t arr_stride[2],
		brechtUnsubmitted Done Inline Actions This stride handling is too complicated. All you need is the size of an element, and you can multiply with that at the very end when calling `test_fn`, using it earlier makes things hard to understand. brecht: This stride handling is too complicated. All you need is the size of an element, and you can…
		mano-wiiAuthorUnsubmitted Done Inline Actions (...) and you can multiply with that at the very end when calling `test_fn`, using it earlier makes things hard to understand. I feel that the patch will have to change a lot if it is to implement another way of dealing with the `offsets`. In this case, the `offset_other` here can be the `x` or the `y`: test_fn(arr + offset_other + offset_iter, user_data) mano-wii: > (...) and you can multiply with that at the very end when calling `test_fn`, using it earlier…
		const int center[2],
		bool (test_fn)(const void arr_item, void *user_data),
		void *user_data)
		{
		const char *arr = arr_v;

		BLI_assert(center[0] >= 0 && center[1] >= 0 && center[0] < arr_shape[0] &&
		center[1] < arr_shape[1]);

		int offset[2] = {center[0] * (int)arr_stride[0], center[1] * (int)arr_stride[1]};
		if (test_fn(arr + offset[0] + offset[1], user_data)) {
		return true;
		}

		int step_max;
		{
		/* Calculate the maximum number of possible steps. */
		int step_max_w, step_max_h;
		if (2 * center[0] < (arr_shape[0] - 1)) {
		step_max_w = 2 * (arr_shape[0] - center[0] - 1);
		}
		else {
		step_max_w = 2 * center[0] + 1;
		}

		if (2 * center[1] < (arr_shape[1] - 1)) {
		step_max_h = 2 * (arr_shape[1] - center[1] - 1) - 1;
		}
		else {
		step_max_h = 2 * center[1];
		}
		step_max = max_ii(step_max_h, step_max_w);
		}

		int step_num = 1;
		int limits[2];
		limits[0] = (arr_shape[0] - 1) * (int)arr_stride[0];
		limits[1] = (int)arr_stride[0] - (int)arr_stride[1];
		bool sign = false;
		while (true) {
		for (int axis = 1; axis >= 0; axis--) {
		int offset_step = (int)arr_stride[axis];
		if (!sign) {
		offset_step *= -1;
		}

		int offset_src = offset[axis] + offset_step;
		int offset_dst = offset[axis] + step_num * offset_step;
		offset[axis] = offset_dst;

		brechtUnsubmitted Done Inline Actions Use start/end rather then src/dst, the latter is usually used to copy something from src to dst, not for iteration. brecht: Use start/end rather then src/dst, the latter is usually used to copy something from src to dst…
		int offset_other = offset[!axis];
		if (offset_other < 0 \|\| offset_other > limits[!axis]) {
		continue;
		}

		if (sign) {
		CLAMP_MIN(offset_src, 0);
		CLAMP_MAX(offset_dst, limits[axis]);
		}
		else {
		CLAMP_MAX(offset_src, limits[axis]);
		CLAMP_MIN(offset_dst, 0);
		}

		while (true) {
		if (test_fn(arr + offset_other + offset_src, user_data)) {
		return true;
		}
		brechtUnsubmitted Done Inline Actions Keep it simple: CLAMP(offset_src, 0, limits[axis]); CLAMP(offset_dst, 0, limits[axis]); brecht: Keep it simple: ``` CLAMP(offset_src, 0, limits[axis]); CLAMP(offset_dst, 0, limits[axis]); ```
		if (offset_src == offset_dst) {
		break;
		}
		offset_src += offset_step;
		}
		}

		if (step_num == step_max) {
		return false;
		}
		step_num++;
		sign = !sign;
		}
		}
		brechtUnsubmitted Done Inline Actions Computing `step_max` in advance looks tricky, can you terminate in a way similar to the old code, but checking both axes? if ((offset[0] < 0 \|\| offset[0] > limits[0]) && (offset[1] < 0 \|\| offset[1] > limits[1])) brecht: Computing `step_max` in advance looks tricky, can you terminate in a way similar to the old…