search

JuliaIO / VideoIO.jl

Reading and writing of video files in Julia via ffmpeg
https://juliaio.github.io/VideoIO.jl/stable
Other
125 stars 53 forks source link

JET fixes/findings #409

Open IanButterworth opened 7 months ago

IanButterworth commented 7 months ago

@galenlynch I've fixed the obvious things and left comments for the less obvious, if you can help.

To run this just

using JET
report_package("VideoIO")
codecov[bot] commented 7 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (e2d0c05) 79.40% compared to head (f0d0a96) 79.40%.

Files Patch % Lines
src/avio.jl 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #409 +/- ## ======================================= Coverage 79.40% 79.40% ======================================= Files 10 10 Lines 1282 1282 ======================================= Hits 1018 1018 Misses 264 264 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

galenlynch commented 7 months ago

Hey I'll fix these soon, thanks for doing this!

IanButterworth commented 4 months ago

low-priority bump :)