Open FalconPDX opened 5 months ago
Attention: 3 lines
in your changes are missing coverage. Please review.
Comparison is base (
89eb1e3
) 83.10% compared to head (f10b034
) 83.00%.
Files | Patch % | Lines |
---|---|---|
audiostream/inc/ni/media/audio/ostream.h | 83.33% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Uses the
pcm
library to encode a range instead of each element in a range. Makes encoding much faster.This is a first of three different packages - starting off with the smallest. The other two (refactor of the ofstream interface + FLAC encoding) will come at a later time in the form of a "stacked PR". This gives users a chance to try out the individual pieces and potentially find issues quicker at the cost of repeated testing.
The other two PRs: ofstream refactor: https://github.com/FalconPDX/ni-media/pull/3 FLAC encoding: https://github.com/FalconPDX/ni-media/pull/4