summaryrefslogtreecommitdiffstats
path: root/options/m_option.h
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2016-08-31 15:25:22 +0200
committerwm4 <wm4@nowhere>2016-08-31 15:31:51 +0200
commitce8dae3f0d9d269fc925f3a83abde4cde3b36ebb (patch)
treea4c795b915502f01b186601c4edaf1b60bd8df40 /options/m_option.h
parentf519887fe7571fb6e05f29b50d6f0bede1646b28 (diff)
downloadmpv-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.h7
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: