Page MenuHome

Further fix for T38726: Rendered viewport was no longer updated after deleting render layers.
ClosedPublic

Authored by Shinsuke Irie (irie) on Feb 23 2014, 6:07 PM.

Diff Detail

Event Timeline

Brecht Van Lommel (brecht) requested changes to this revision.Feb 24 2014, 2:59 PM
Brecht Van Lommel (brecht) added inline comments.
source/blender/render/intern/source/pipeline.c
505–506 ↗(On Diff #1099)

This is too ugly a hack, there has to be a better solution.

source/blender/render/intern/source/pipeline.c
505–506 ↗(On Diff #1099)

Agreed, this should be replaced with the better solution. But maybe it becomes complicated.

Shinsuke Irie (irie) updated this revision to Unknown Object (????).Feb 25 2014, 6:32 PM
  • Add a numerical identifier to RenderData to ensure uniqueness of each edit in render layer list.

Don't compare the whole parameters anymore.
The preview might not be updated correctly if incrementation of the
identifier overflows, though that's quite rare case.

BTW, the previous patch should've used SceneRenderLayer, not RenderLayer.