Hi,

On Mon, May 30, 2011 at 1:02 PM, Stefano Sabatini
<[email protected]> wrote:
> On date Monday 2011-05-30 21:15:17 +0200, Anton Khirnov encoded:
>> ---
>>  doc/APIchanges         |    4 ++
>>  libavformat/avformat.h |   35 ++++++-------
>>  libavformat/metadata.c |  121 ++++-------------------------------------
>>  libavformat/metadata.h |   12 +----
>>  libavutil/Makefile     |    3 +
>>  libavutil/avutil.h     |    2 +-
>>  libavutil/meta.h       |   78 ++++++++++++++++++++++++++
>>  libavutil/metadata.c   |  142 
>> ++++++++++++++++++++++++++++++++++++++++++++++++
>>  libavutil/metadata.h   |   44 +++++++++++++++
>>  9 files changed, 301 insertions(+), 140 deletions(-)
>>  create mode 100644 libavutil/meta.h
>>  create mode 100644 libavutil/metadata.c
>>  create mode 100644 libavutil/metadata.h
>>
>> diff --git a/doc/APIchanges b/doc/APIchanges
>> index 853975e..d37b2fb 100644
>> --- a/doc/APIchanges
>> +++ b/doc/APIchanges
>> @@ -13,6 +13,10 @@ libavutil:   2011-04-18
>>
>>  API changes, most recent first:
>>
>> +2011-05-xx - xxxxxxx - lavu 51.2.0 - AVMetadata
>> +  Move AVMetadata from lavf to lavu -- new installed headers
>> +  meta.h (accessing metadata) and metadata.h (internals).
>
> Sorry to bother with more terminology pedantry, but I suggest
> avmetadata.h for the public API, and avmetadata_internal.h for the
> internal interface

We don't do that for any other pieces in lavu...?

> and keep everywhere the old interface and naming
> scheme, which I find more clear and consistent, by using the ld gnu
> (dirty) trick.

This I agree with. No renames needed, we've moved API without renames before.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to