Closed galenlynch closed 4 years ago
Merging #258 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #258 +/- ##
=======================================
Coverage 77.09% 77.09%
=======================================
Files 14 14
Lines 598 598
=======================================
Hits 461 461
Misses 137 137
Impacted Files | Coverage Δ | |
---|---|---|
src/avio.jl | 79.15% <ø> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d2166e5...4f3e9b8. Read the comment docs.
Two fields of
VideoReader
,framerate
andaspect_ratio
, are both of typeRational
, an abstract type. In reality, the constructor ofVideoReader
sets both of these fields to be typeRational{Cint}
. By changing the type definition to reflect that, it reduces type instability in code accessingVideoReader
fields, while not restricting the generality ofVideoReader
.