diff options
author | wm4 <wm4@nowhere> | 2016-08-31 15:25:22 +0200 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2016-08-31 15:31:51 +0200 |
commit | ce8dae3f0d9d269fc925f3a83abde4cde3b36ebb (patch) | |
tree | a4c795b915502f01b186601c4edaf1b60bd8df40 /options/m_option.h | |
parent | f519887fe7571fb6e05f29b50d6f0bede1646b28 (diff) | |
download | mpv-ce8dae3f0d9d269fc925f3a83abde4cde3b36ebb.tar.bz2 mpv-ce8dae3f0d9d269fc925f3a83abde4cde3b36ebb.tar.xz |
m_property: remove pointless explicitly clamping
This is basically dead code, and even the commit that added this code 4
years ago said that this should be for debugging only.
(Though it is possible that the clamp callback was used for something
else, and then unused again. Also, some of the clamping code remains and
is used for internal checking, e.g. clamp_double().)
Diffstat (limited to 'options/m_option.h')
-rw-r--r-- | options/m_option.h | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/options/m_option.h b/options/m_option.h index 847cd1097c..bf7ee5939d 100644 --- a/options/m_option.h +++ b/options/m_option.h @@ -291,13 +291,6 @@ struct m_option_type { // to the valid value range of the option. void (*multiply)(const m_option_t *opt, void *val, double f); - // Clamp the value in val to the option's valid value range. - // Return values: - // M_OPT_OUT_OF_RANGE: val was invalid, and modified (clamped) to be valid - // M_OPT_INVALID: val was invalid, and can't be made valid - // 0: val was already valid and is unchanged - int (*clamp)(const m_option_t *opt, void *val); - // Set the option value in dst to the contents of src. // (If the option is dynamic, the old value in *dst has to be freed.) // Return values: |