--- videoeffect.cpp (revision 689013) +++ videoeffect.cpp (working copy) @@ -170,7 +170,8 @@ QVariant VideoEffect::parameterValue(int Q_ASSERT(m_pluginParams); api->get_parameters(post, m_pluginParams); int i = 0; - for (; i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST; ++i); + while (i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST) + ++i; if (i == parameterIndex) { xine_post_api_parameter_t &p = desc->parameter[i]; switch (p.type) { @@ -211,7 +212,8 @@ void VideoEffect::setParameterValue(int xine_post_api_descr_t *desc = api->get_param_descr(); Q_ASSERT(m_pluginParams); int i = 0; - for (; i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST; ++i); + while (i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST) + ++i; if (i == parameterIndex) { xine_post_api_parameter_t &p = desc->parameter[i]; switch (p.type) { Index: audioeffect.cpp =================================================================== --- audioeffect.cpp (revision 689013) +++ audioeffect.cpp (working copy) @@ -169,7 +169,8 @@ QVariant AudioEffect::parameterValue(int Q_ASSERT(m_pluginParams); api->get_parameters(post, m_pluginParams); int i = 0; - for (; i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST; ++i); + while (i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST) + ++i; if (i == parameterIndex) { xine_post_api_parameter_t &p = desc->parameter[i]; switch (p.type) { @@ -210,7 +211,8 @@ void AudioEffect::setParameterValue(int xine_post_api_descr_t *desc = api->get_param_descr(); Q_ASSERT(m_pluginParams); int i = 0; - for (; i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST; ++i); + while (i < parameterIndex && desc->parameter[i].type != POST_PARAM_TYPE_LAST) + ++i; if (i == parameterIndex) { xine_post_api_parameter_t &p = desc->parameter[i]; switch (p.type) {