On Mon, 2016-02-29 at 15:46 +0530, [email protected] wrote: > - pa_memblockq_seek(s->memblockq, delta * (int64_t) > s->rtp_context.frame_size, PA_SEEK_RELATIVE, true); > + pa_memblockq_seek(s->memblockq, delta * (int64_t) > pa_rtp_context_get_frame_size(s->rtp_context), PA_SEEK_RELATIVE, > + true);
The context's frame size is the same as pa_frame_size(&s->sdp_info.sample_spec), so it's not really necessary to query it from the context. To avoid the pa_frame_size() call too, I'd add a frame_size variable to the session struct. (This is not a big deal, though; I'm ok with the code as it is.) > - pa_rtp_context_destroy(&s->rtp_context); > + pa_rtp_context_destroy(s->rtp_context); You renamed pa_rtp_context_init() to pa_rtp_context_new(). I think it would be good to also rename pa_rtp_context_destroy() to pa_rtp_context_free(). -- Tanu _______________________________________________ pulseaudio-discuss mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
