blender
/
blender
136
404
Fork 586
Code Issues 6.2k Pull Requests 754 Projects 19 Wiki Activity

Cleanup: use boolean arguments and return values

This commit is contained in:
Campbell Barton 2022-04-26 14:25:58 +10:00
parent a9d1b3d7e3
commit 6d12bc9e91
10 changed files with 50 additions and 51 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
source/blender/blenkernel/BKE_particle.h
View File

@ -391,12 +391,12 @@ void psys_cache_child_paths(struct ParticleSimulationData *sim,
float cfra,
bool editupdate,
bool use_render_params);
int do_guides(struct Depsgraph *depsgraph,
struct ParticleSettings *part,
struct ListBase *effectors,
ParticleKey *state,
int index,
float time);
bool do_guides(struct Depsgraph *depsgraph,
struct ParticleSettings *part,
struct ListBase *effectors,
ParticleKey *state,
int index,
float time);
void precalc_guides(struct ParticleSimulationData *sim, struct ListBase *effectors);
float psys_get_timestep(struct ParticleSimulationData *sim);
float psys_get_child_time(struct ParticleSystem *psys,

12
source/blender/blenkernel/intern/particle.c
View File

@ -2390,12 +2390,12 @@ void precalc_guides(ParticleSimulationData *sim, ListBase *effectors)
}
}
int do_guides(Depsgraph *depsgraph,
ParticleSettings *part,
ListBase *effectors,
ParticleKey *state,
int index,
float time)
bool do_guides(Depsgraph *depsgraph,
ParticleSettings *part,
ListBase *effectors,
ParticleKey *state,
int index,
float time)
{
CurveMapping *clumpcurve = (part->child_flag & PART_CHILD_USE_CLUMP_CURVE) ? part->clumpcurve :
NULL;

17
source/blender/blenkernel/intern/pointcache.c
View File

@ -1399,7 +1399,8 @@ static size_t ptcache_filename_ext_append(PTCacheID *pid,
return len;
}
static int ptcache_filename(PTCacheID *pid, char *filename, int cfra, short do_path, short do_ext)
static int ptcache_filename(
PTCacheID *pid, char *filename, int cfra, const bool do_path, const bool do_ext)
{
int len = 0;
char *idname;
@ -1464,7 +1465,7 @@ static PTCacheFile *ptcache_file_open(PTCacheID *pid, int mode, int cfra)
}
}
ptcache_filename(pid, filename, cfra, 1, 1);
ptcache_filename(pid, filename, cfra, true, true);
if (mode == PTCACHE_FILE_READ) {
fp = BLI_fopen(filename, "rb");
@ -2630,7 +2631,7 @@ void BKE_ptcache_id_clear(PTCacheID *pid, int mode, unsigned int cfra)
return;
}
len = ptcache_filename(pid, filename, cfra, 0, 0); /* no path */
len = ptcache_filename(pid, filename, cfra, false, false); /* no path */
/* append underscore terminator to ensure we don't match similar names
* from objects whose names start with the same prefix
*/
@ -2712,7 +2713,7 @@ void BKE_ptcache_id_clear(PTCacheID *pid, int mode, unsigned int cfra)
case PTCACHE_CLEAR_FRAME:
if (pid->cache->flag & PTCACHE_DISK_CACHE) {
if (BKE_ptcache_id_exist(pid, cfra)) {
ptcache_filename(pid, filename, cfra, 1, 1); /* no path */
ptcache_filename(pid, filename, cfra, true, true); /* no path */
BLI_delete(filename, false, false);
}
}
@ -2753,7 +2754,7 @@ bool BKE_ptcache_id_exist(PTCacheID *pid, int cfra)
if (pid->cache->flag & PTCACHE_DISK_CACHE) {
char filename[MAX_PTCACHE_FILE];
ptcache_filename(pid, filename, cfra, 1, 1);
ptcache_filename(pid, filename, cfra, true, true);
return BLI_exists(filename);
}
@ -3500,7 +3501,7 @@ void BKE_ptcache_disk_cache_rename(PTCacheID *pid, const char *name_src, const c
/* get "from" filename */
BLI_strncpy(pid->cache->name, name_src, sizeof(pid->cache->name));
len = ptcache_filename(pid, old_filename, 0, 0, 0); /* no path */
len = ptcache_filename(pid, old_filename, 0, false, false); /* no path */
ptcache_path(pid, path);
dir = opendir(path);
@ -3522,7 +3523,7 @@ void BKE_ptcache_disk_cache_rename(PTCacheID *pid, const char *name_src, const c
if (frame != -1) {
BLI_join_dirfile(old_path_full, sizeof(old_path_full), path, de->d_name);
ptcache_filename(pid, new_path_full, frame, 1, 1);
ptcache_filename(pid, new_path_full, frame, true, true);
BLI_rename(old_path_full, new_path_full);
}
}
@ -3555,7 +3556,7 @@ void BKE_ptcache_load_external(PTCacheID *pid)
ptcache_path(pid, path);
len = ptcache_filename(pid, filename, 1, 0, 0); /* no path */
len = ptcache_filename(pid, filename, 1, false, false); /* no path */
dir = opendir(path);
if (dir == NULL) {

2
source/blender/editors/sculpt_paint/paint_vertex.cc
View File

@ -412,7 +412,7 @@ static float wpaint_blend(const VPaint *wp,
const float alpha,
float paintval,
const float UNUSED(brush_alpha_value),
const short do_flip)
const bool do_flip)
{
const Brush *brush = wp->paint.brush;
IMB_BlendMode blend = (IMB_BlendMode)brush->blend;

2
source/blender/geometry/intern/uv_parametrizer.c
View File

@ -203,7 +203,7 @@ typedef struct PHandle {
RNG *rng;
float blend;
char do_aspect;
bool do_aspect;
} PHandle;
/* PHash

6
source/blender/imbuf/intern/rectop.c
View File

@ -515,7 +515,7 @@ void IMB_rectblend(ImBuf *dbuf,
const unsigned short *cmaskrect = curvemask, *cmr;
unsigned short *dmaskrect = dmask, *dmr;
const unsigned short *texmaskrect = texmask, *tmr;
int do_float, do_char, srcskip, destskip, origskip, x;
int srcskip, destskip, origskip, x;
IMB_blend_func func = NULL;
IMB_blend_func_float func_float = NULL;
@ -535,8 +535,8 @@ void IMB_rectblend(ImBuf *dbuf,
return;
}
do_char = (sbuf && sbuf->rect && dbuf->rect && obuf->rect);
do_float = (sbuf && sbuf->rect_float && dbuf->rect_float && obuf->rect_float);
const bool do_char = (sbuf && sbuf->rect && dbuf->rect && obuf->rect);
const bool do_float = (sbuf && sbuf->rect_float && dbuf->rect_float && obuf->rect_float);
if (do_char) {
drect = dbuf->rect + ((size_t)desty) * dbuf->x + destx;

22
source/blender/imbuf/intern/scaling.c
View File

@ -171,14 +171,13 @@ static void imb_half_y_no_alloc(struct ImBuf *ibuf2, struct ImBuf *ibuf1)
uchar *p1, *p2, *_p1, *dest;
short a, r, g, b;
int x, y;
int do_rect, do_float;
float af, rf, gf, bf, *p1f, *p2f, *_p1f, *destf;
p1 = p2 = NULL;
p1f = p2f = NULL;
do_rect = (ibuf1->rect != NULL);
do_float = (ibuf1->rect_float != NULL && ibuf2->rect_float != NULL);
const bool do_rect = (ibuf1->rect != NULL);
const bool do_float = (ibuf1->rect_float != NULL && ibuf2->rect_float != NULL);
_p1 = (uchar *)ibuf1->rect;
dest = (uchar *)ibuf2->rect;
@ -264,7 +263,6 @@ struct ImBuf *IMB_double_fast_y(struct ImBuf *ibuf1)
int *p1, *dest1, *dest2;
float *p1f, *dest1f, *dest2f;
int x, y;
int do_rect, do_float;
if (ibuf1 == NULL) {
return NULL;
@ -273,8 +271,8 @@ struct ImBuf *IMB_double_fast_y(struct ImBuf *ibuf1)
return NULL;
}
do_rect = (ibuf1->rect != NULL);
do_float = (ibuf1->rect_float != NULL);
const bool do_rect = (ibuf1->rect != NULL);
const bool do_float = (ibuf1->rect_float != NULL);
ibuf2 = IMB_allocImBuf(ibuf1->x, 2 * ibuf1->y, ibuf1->planes, ibuf1->flags);
if (ibuf2 == NULL) {
@ -358,8 +356,8 @@ MINLINE void premul_ushort_to_straight_uchar(unsigned char *result, const unsign
void imb_onehalf_no_alloc(struct ImBuf *ibuf2, struct ImBuf *ibuf1)
{
int x, y;
const short do_rect = (ibuf1->rect != NULL);
const short do_float = (ibuf1->rect_float != NULL) && (ibuf2->rect_float != NULL);
const bool do_rect = (ibuf1->rect != NULL);
const bool do_float = (ibuf1->rect_float != NULL) && (ibuf2->rect_float != NULL);
if (do_rect && (ibuf2->rect == NULL)) {
imb_addrectImBuf(ibuf2);
@ -892,8 +890,8 @@ static bool q_scale_linear_interpolation(struct ImBuf *ibuf, int newx, int newy)
static ImBuf *scaledownx(struct ImBuf *ibuf, int newx)
{
const int do_rect = (ibuf->rect != NULL);
const int do_float = (ibuf->rect_float != NULL);
const bool do_rect = (ibuf->rect != NULL);
const bool do_float = (ibuf->rect_float != NULL);
const size_t rect_size = IMB_get_rect_len(ibuf) * 4;
uchar *rect, *_newrect, *newrect;
@ -1033,8 +1031,8 @@ static ImBuf *scaledownx(struct ImBuf *ibuf, int newx)
static ImBuf *scaledowny(struct ImBuf *ibuf, int newy)
{
const int do_rect = (ibuf->rect != NULL);
const int do_float = (ibuf->rect_float != NULL);
const bool do_rect = (ibuf->rect != NULL);
const bool do_float = (ibuf->rect_float != NULL);
const size_t rect_size = IMB_get_rect_len(ibuf) * 4;
uchar *rect, *_newrect, *newrect;

2
source/blender/python/BPY_extern.h
View File

@ -67,7 +67,7 @@ void BPY_text_free_code(struct Text *text);
void BPY_modules_update(void);
void BPY_modules_load_user(struct bContext *C);
void BPY_app_handlers_reset(short do_all);
void BPY_app_handlers_reset(bool do_all);
/**
* Update function, it gets rid of python-drivers global dictionary: `bpy.app.driver_namespace`,

2
source/blender/python/intern/bpy_app_handlers.c
View File

@ -244,7 +244,7 @@ PyObject *BPY_app_handlers_struct(void)
return ret;
}
void BPY_app_handlers_reset(const short do_all)
void BPY_app_handlers_reset(const bool do_all)
{
PyGILState_STATE gilstate;
int pos = 0;

24
source/blender/render/intern/pipeline.c
View File

@ -155,12 +155,12 @@ static void render_callback_exec_id(Render *re, Main *bmain, ID *id, eCbEvent ev
/** \name Allocation & Free
* \{ */
static int do_write_image_or_movie(Render *re,
Main *bmain,
Scene *scene,
bMovieHandle *mh,
const int totvideos,
const char *name_override);
static bool do_write_image_or_movie(Render *re,
Main *bmain,
Scene *scene,
bMovieHandle *mh,
const int totvideos,
const char *name_override);
/* default callbacks, set in each new render */
static void result_nothing(void *UNUSED(arg), RenderResult *UNUSED(rr))
@ -2020,12 +2020,12 @@ bool RE_WriteRenderViewsMovie(ReportList *reports,
return ok;
}
static int do_write_image_or_movie(Render *re,
Main *bmain,
Scene *scene,
bMovieHandle *mh,
const int totvideos,
const char *name_override)
static bool do_write_image_or_movie(Render *re,
Main *bmain,
Scene *scene,
bMovieHandle *mh,
const int totvideos,
const char *name_override)
{
char name[FILE_MAX];
RenderResult rres;