# HG changeset patch # User ib # Date 1398344918 0 # Node ID 1bcac20d33932f6f5737040efb2de1751aa0c725 # Parent 3ec195585bfd8dbb3262221542e16174237f3953 Rewrite add_vf(). Avoid confusing usage of i = -1 to indicate that a video filter has been found. Check vf_settings[i].name instead. As a result, the index of the video filter found won't be destroyed. diff -r 3ec195585bfd -r 1bcac20d3393 gui/interface.c --- a/gui/interface.c Thu Apr 24 12:58:40 2014 +0000 +++ b/gui/interface.c Thu Apr 24 13:08:38 2014 +0000 @@ -281,13 +281,13 @@ int i = 0; while (vf_settings[i].name) { - if (strcmp(vf_settings[i++].name, vf) == 0) { - i = -1; + if (strcmp(vf_settings[i].name, vf) == 0) break; - } + + i++; } - if (i != -1) { + if (!vf_settings[i].name) { void *settings = realloc(vf_settings, (i + 2) * sizeof(m_obj_settings_t)); if (!settings)