[FFmpeg-devel] [PATCH 07/10] swr: start doxycommenting the API.
Clément Bœsch
ubitux at gmail.com
Thu Nov 17 10:22:02 CET 2011
On Thu, Nov 17, 2011 at 02:59:35AM +0100, Michael Niedermayer wrote:
[...]
> > + void *log_ctx; ///< logging context
>
> parent loging context
>
Fixed.
>
> > + enum AVSampleFormat in_sample_fmt; ///< input sample format
> > + enum AVSampleFormat int_sample_fmt; ///< internal sample format (AV_SAMPLE_FMT_FLT or AV_SAMPLE_FMT_S16)
> > + enum AVSampleFormat out_sample_fmt; ///< output sample format
> > + int64_t in_ch_layout; ///< input channel layout
> > + int64_t out_ch_layout; ///< output channel layout
> > + int in_sample_rate; ///< input sample rate
> > + int out_sample_rate; ///< output sample rate
> > + int flags; ///< miscellaneous flags such as SWR_FLAG_RESAMPLE
> > + float slev; ///< surround mixing level, XXX: unused
> > + float clev; ///< center mixing level
>
> > + float rematrix_volume; ///< used to set a different rematrixing volume than 1.0
>
> rematrixing sometimes has to reduce the volume to avoid overflow so 1.0 is not always teh default
>
OK, changed to "rematrix volume coefficient"
>
> > + const int *channel_map; ///< channel index (or -1 if muted channel) map
> > + int used_ch_count; ///< number of used channels (mapped channel count if channel_map, otherwise in.ch_count)
>
> number of used input channels
>
Indeed, added the "input".
>
> [...]
>
> > + float matrix [SWR_CH_MAX][SWR_CH_MAX ]; ///< float channel levels squared matrix for rematrixing
> > + int32_t matrix32 [SWR_CH_MAX][SWR_CH_MAX ]; ///< integer channel levels squared matrix for rematrixing
> > + uint8_t matrix_ch[SWR_CH_MAX][SWR_CH_MAX+1]; ///< channel matrix for rematrixing
>
> these sound non ideal, i suggest to leave it to a later patch
>
OK, feel free to add something that makes sense to you then, I'm curious
now :)
> the rest sounded ok to me
>
Great! I also removed the "/* private fields */" comment (previously
"//below are private") because it doesn't really make sense: the struct is
opaque so all the fields are actually private.
I just pused the whole patch, thanks for the review.
--
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111117/88aebe3b/attachment.asc>
More information about the ffmpeg-devel
mailing list