Use invariant culture for tonemap options (#12991)

pull/13026/head
gnattu 4 months ago committed by GitHub
parent 25321d7f80
commit d292fde9e2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

@ -3527,20 +3527,29 @@ namespace MediaBrowser.Controller.MediaEncoding
{ {
// tonemapx requires yuv420p10 input for dovi reshaping, let ffmpeg convert the frame when necessary // tonemapx requires yuv420p10 input for dovi reshaping, let ffmpeg convert the frame when necessary
var tonemapFormat = requireDoviReshaping ? "yuv420p" : outFormat; var tonemapFormat = requireDoviReshaping ? "yuv420p" : outFormat;
var tonemapArgString = "tonemapx=tonemap={0}:desat={1}:peak={2}:t=bt709:m=bt709:p=bt709:format={3}";
var tonemapArgs = $"tonemapx=tonemap={options.TonemappingAlgorithm}:desat={options.TonemappingDesat}:peak={options.TonemappingPeak}:t=bt709:m=bt709:p=bt709:format={tonemapFormat}";
if (options.TonemappingParam != 0) if (options.TonemappingParam != 0)
{ {
tonemapArgs += $":param={options.TonemappingParam}"; tonemapArgString += ":param={4}";
} }
var range = options.TonemappingRange; var range = options.TonemappingRange;
if (range == TonemappingRange.tv || range == TonemappingRange.pc) if (range == TonemappingRange.tv || range == TonemappingRange.pc)
{ {
tonemapArgs += $":range={options.TonemappingRange}"; tonemapArgString += ":range={5}";
} }
var tonemapArgs = string.Format(
CultureInfo.InvariantCulture,
tonemapArgString,
options.TonemappingAlgorithm,
options.TonemappingDesat,
options.TonemappingPeak,
tonemapFormat,
options.TonemappingParam,
options.TonemappingRange);
mainFilters.Add(tonemapArgs); mainFilters.Add(tonemapArgs);
} }
else else

Loading…
Cancel
Save