diff options
author | Peter Schlaile <peter@schlaile.de> | 2006-06-10 23:56:28 +0400 |
---|---|---|
committer | Peter Schlaile <peter@schlaile.de> | 2006-06-10 23:56:28 +0400 |
commit | c4229b0272ac223e3723773321ce9263ef858607 (patch) | |
tree | 1f9ea6ba659b09eac7532924b3591aa3770242d2 /release/plugins/sequence | |
parent | 43e776690f651f941b2612d94a943398e3518de2 (diff) |
==Sequencer==
Fixed the blur-plugin (and maybe a lot more) crashes by expecting
future float-buffer aware sequencer-plugins to have a bumped PLUGIN_VERSION
number. Since quality and speed is degraded by converting the float
buffer first to byte, performing the effect on bytes and then converting
back again an additional warning is displayed in the effect strip,
suggesting to update the used sequencer-plugins.
Fixed some more crashes along the way.
Float buffer aware sequencer plugins should
- first check, if the output-ibuf has a rect_float
=> perform all operations with floats (input and output)
- if not: perform everything on bytes (intput and output)
Diffstat (limited to 'release/plugins/sequence')
-rw-r--r-- | release/plugins/sequence/blur.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/release/plugins/sequence/blur.c b/release/plugins/sequence/blur.c index 975a0c8df93..9687d04e066 100644 --- a/release/plugins/sequence/blur.c +++ b/release/plugins/sequence/blur.c @@ -115,7 +115,7 @@ void blurbuf(struct ImBuf *ibuf, int nr, Cast *cast) } if(tbuf->x<4 || tbuf->y<4) break; } - + /* enlarge */ for(i=0; i<nr; i++) { ttbuf = double_x(tbuf); @@ -137,6 +137,7 @@ void blurbuf(struct ImBuf *ibuf, int nr, Cast *cast) if(cast->gamma != 1.0) gamwarp(tbuf, 1.0 / cast->gamma); + /* Very bad code warning! This fails badly with float-buffers!!! */ freeN(ibuf->rect); ibuf->rect= tbuf->rect; freeN(tbuf); |