Differential D2097

Consolidate multiple checks for out->rect_float in prepare_effect_imbufs()
ClosedPublic
Actions

Authored by Chad Fraleigh (chadf) on Jul 9 2016, 6:04 AM.

Details

Reviewers

Commits

rBS17e40f821e56: Consolidate multiple checks for out->rect_float in prepare_effect_imbufs()
rB17e40f821e56: Consolidate multiple checks for out->rect_float in prepare_effect_imbufs()

Summary

Many checks for out->rect_float being [non-]NULL are done back-to-back. Combining them into a single check for slightly more efficient code and less code clutter for easier readability/understanding.

Diff Detail

Repository: rB Blender

Event Timeline

Chad Fraleigh (chadf) updated this revision to Diff 7037.Jul 9 2016, 6:04 AM

Chad Fraleigh (chadf) retitled this revision from to Consolidate multiple checks for out->rect_float in prepare_effect_imbufs().

Chad Fraleigh (chadf) updated this object.

Chad Fraleigh (chadf) set the repository for this revision to rB Blender.

Sergey Sharybin (sergey) accepted this revision.Jul 14 2016, 2:17 PM

Sergey Sharybin (sergey) added a reviewer: Sergey Sharybin (sergey).

This revision is now accepted and ready to land.Jul 14 2016, 2:17 PM

Closed by commit rB17e40f821e56: Consolidate multiple checks for out->rect_float in prepare_effect_imbufs() (authored by Chad Fraleigh (chadf), committed by Sergey Sharybin (sergey)). · Explain WhyJul 14 2016, 2:17 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

source/

blender/

blenkernel/

intern/

seqeffects.c

44 lines

Diff 7054

View Options

source/blender/blenkernel/intern/seqeffects.c

Consolidate multiple checks for out->rect_float in prepare_effect_imbufs()ClosedPublicActions