Differential D1549
Fix T46223: multiview image sequences crashing. Authored by Brecht Van Lommel (brecht) on Oct 11 2015, 3:33 AM. Tags None Subscribers None
Details
The IMA_IS_STEREO and IMA_IS_MULTIVIEW flags can get out of sync with the Also fixes memory leak when loading multilayer multiview images.
Diff Detail
Event TimelineComment Actions @Dalai Felinto (dfelinto), can you think of any reason these flags should exist, or do you agree it's just redundant state? The fix seems to work well, but it's not so easy to undo this commit once it has been made, so I would like your opinion as on this. Comment Actions The flags can be dropped with no problems. They were there mostly for performance and quick look up if I remember correctly. I was just waiting to test this patch over different situations. But, assuming you tested it well, if this introduce any bug in multiview we can fix afterwards. So go ahead if you want (otherwise I should be able to test it on Thursday) |