cmxl / FFmpeg.NET

.NET wrapper for common ffmpeg tasks
MIT License
611 stars 99 forks source link

Decimal separator in durations #27

Closed ronaldvdv closed 5 years ago

ronaldvdv commented 5 years ago

Currently, FFmpeg.NET is parsing durations from metadata using TryParse without specifying an IFormatProvider. On machines with a non-default locale (e.g. having comma as a decimal separator), this will lead to a mismatch with FFmpeg which still uses the universal dot as a separator. Instead, we should use CultureInfo.InvariantCulture. Will open a pull request for this.

ronaldvdv commented 5 years ago

Duplicate of #23