Skip to content
Snippets Groups Projects

tone_mapping: Fix inferred default tone map param

Closed quietvoid requested to merge quietvoid/libplacebo:tone_map_infer_clamp_order into master

When the param is not set or zero, the default should be used before clamping, otherwise the param is left to the clamped minimum and the default param is never set.

Fixes: https://github.com/mpv-player/mpv/issues/11076

I'm not sure how to handle intentional param overrides to 0, the current code would default to the min instead.
I think defaulting to sane values makes sense.

Edited by quietvoid

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading