diff options
author | Mike Erwin <significant.bit@gmail.com> | 2016-07-22 04:31:37 +0300 |
---|---|---|
committer | Mike Erwin <significant.bit@gmail.com> | 2016-07-22 09:17:52 +0300 |
commit | 2b77b1ce465ab09052567a58d1b9f6a014a563cd (patch) | |
tree | c7028d37887cd3b64b8e11f0cd5f3b7bda10a63c /source/blender/render | |
parent | bd52875dd67ea907273730c686d849828240e7b3 (diff) |
remove double-checked conditions
Checking a condition right after we’ve checked it (and it hasn’t
changed). Most of these are trivial to understand.
split_quads in convertblender.c:
It seems quads should be processed and triangles should be marked as
not needing split. So I removed the outer vlr->v4 check.
Found with PVS-Studio T48917
Diffstat (limited to 'source/blender/render')
-rw-r--r-- | source/blender/render/intern/source/convertblender.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/blender/render/intern/source/convertblender.c b/source/blender/render/intern/source/convertblender.c index ab828a0c04e..86961cdd169 100644 --- a/source/blender/render/intern/source/convertblender.c +++ b/source/blender/render/intern/source/convertblender.c @@ -4192,7 +4192,7 @@ static void split_quads(ObjectRen *obr, int dir) vlr= RE_findOrAddVlak(obr, a); /* test if rendering as a quad or triangle, skip wire */ - if (vlr->v4 && (vlr->flag & R_STRAND)==0 && (vlr->mat->material_type != MA_TYPE_WIRE)) { + if ((vlr->flag & R_STRAND)==0 && (vlr->mat->material_type != MA_TYPE_WIRE)) { if (vlr->v4) { |