Increase precision of frame rate property
ClosedPublic

Authored by astippich on Feb 17 2019, 7:31 PM.

Details

Summary

Some video files have frame rates
of 23.976 or similar, requiring the representation
as double. The data will be displayed
with two digit precision, as done by
kdenlive for example.

Diff Detail

Repository
R286 KFileMetaData
Branch
ffmpeg_framerate
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 8566
Build 8584: arc lint + arc unit
astippich created this revision.Feb 17 2019, 7:31 PM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptFeb 17 2019, 7:31 PM
Restricted Application added subscribers: Baloo, kde-frameworks-devel. · View Herald Transcript
astippich requested review of this revision.Feb 17 2019, 7:31 PM
bruns added inline comments.Feb 17 2019, 8:28 PM
autotests/ffmpegextractortest.cpp
50

You can probably use (24.0/1001) for the exact value.

astippich updated this revision to Diff 52103.Feb 19 2019, 7:24 PM
  • rebase on master
  • use exact value in test
bruns requested changes to this revision.Feb 19 2019, 7:35 PM
bruns added inline comments.
This revision now requires changes to proceed.Feb 19 2019, 7:35 PM
astippich updated this revision to Diff 52109.Feb 19 2019, 8:20 PM
  • do not use QString::number
astippich marked 2 inline comments as done.Feb 19 2019, 8:21 PM
bruns accepted this revision.Feb 19 2019, 11:08 PM
This revision is now accepted and ready to land.Feb 19 2019, 11:08 PM
This revision was automatically updated to reflect the committed changes.