RepoMirrors
/
mpv
mirror of https://github.com/mpv-player/mpv
1
0
Fork 0
Code Issues Releases Wiki Activity

m_option: clamp integer before adding a value 

This is for the previous commit, and should affect behavior with the
special M_PROPERTY_GET_CONSTRICTED_TYPE mechanism only. The effect is
that cycling the "edition" property, if the option is set to "auto",
will change to the second edition instead of the first.

Normally, option values must always be within their range, so this
should not affect anything else. M_PROPERTY_GET_CONSTRICTED_TYPE is
sort-of fine with this kind of behavior.

If this affects any other M_PROPERTY_GET_CONSTRICTED_TYPE users
neqatively, I will revert the change.
This commit is contained in:
wm4 2019-12-16 01:52:52 +01:00
parent aa5f234b43
commit de0f9b9f2d
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
options/m_option.c
View File

@ -282,6 +282,8 @@ static void add_int64(const m_option_t *opt, void *val, double add, bool wrap)
{ {
int64_t v = *(int64_t *)val; int64_t v = *(int64_t *)val;
clamp_int64(opt, &v);
v = v + add; v = v + add;
bool is64 = opt->type->size == sizeof(int64_t); bool is64 = opt->type->size == sizeof(int64_t);