[asterisk-dev] Re: [asterisk-commits] trunk r32255 - in /trunk: include/asterisk/rtp.h rtp.c

[Kevin P. Fleming]

> Author: oej
> Date: Mon Jun  5 04:42:28 2006
> New Revision: 32255
> 
> URL: http://svn.digium.com/view/asterisk?rev=32255&view=rev
> Log:
> Move definition of RTP structure to rtp.h (preparing for future
> changes)

I don't think this is an acceptable change. Any modules that need information out of the RTP structure should call functions exported by rtp.c to get it, following the model that already exists. This is an internal 'opaque' structure entirely for use by rtp.c.

-- 
Kevin P. Fleming
Senior Software Engineer
Digium, Inc.