[Ffmpeg-devel] [PATCH 12/31] Move rtp_parse_open() to the header file.
Wed Mar 14 14:19:58 CET 2007
-----BEGIN PGP SIGNED MESSAGE-----
Michael Niedermayer schreef:
> On Mon, Mar 12, 2007 at 04:21:33PM +0100, Panagiotis Issaris wrote:
>> takis.issaris at uhasselt.be schreef:
>>> Move libavutil/rtp.c public API comments to rtp.h. The moved comments are
>>> Signed-off-by: Panagiotis Issaris <takis.issaris at uhasselt.be>
>>> libavformat/rtp.c | 6 ------
>>> libavformat/rtp.h | 7 +++++++
>>> 2 files changed, 7 insertions(+), 6 deletions(-)
>> Ping... :-)
>> All other patches from the [xx/31] patchset have been reviewed and
>> applied, the only exceptions which received no comments were the ones
>> listed here:
>> [12/31] Move rtp_parse_open() to the header file.
>> [13/31] Move rtp_parse_packet() to the header file.
>> [14/31] Move rtp_set_remote_url() to the header file.
>> [15/31] Move rtp_get_local_port() to the header file.
>> [16/31] Move rtp_get_file_handles() to the header file.
>> The descriptions are incorrect as the function itself isn't moved, but
>> the Doxygen comments only.
> they move comments to rtp.h yes, that on its own would be all fine but
> rtp.h is a installed header and these functions dont look like they
> should be in a installed header
Okay, if you wouldn't mind I would still prefer to move them to the
header file (as long as a header file exists I think it is still useful).
> maybe rtp.h should not be installed at all i dunno
Oh, I see. I'll start a new thread to ask other for their opinion.
With friendly regards,
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
-----END PGP SIGNATURE-----
More information about the ffmpeg-devel