[FFmpeg-soc] [soc] libavsequencer [PATCH 01/08] Music module public API header file.
Sebastian Vater
cdgs.basty at googlemail.com
Sat Jul 10 15:11:03 CEST 2010
Sebastian Vater a écrit :
> Stefano Sabatini a écrit :
>
>
>>
>>
>>> +typedef struct AVSequencerModule {
>>> + /** Metadata information: Original module file name, module name,
>>> + * module message, artist, genre, album, begin and finish date of
>>> + * composition and comment. */
>>> + AVSequencerMetadata *metadata;
>>>
>>>
>> AVSequencerMetadata is not shown in your patches, also I wonder, won't
>> be this a repetition of libavformat AVMetaData?
>>
Fixed by globally replacing AVSequencerMetadata to AVMetadata, I decided
it's way better to implement the new tags in AVMetadata itself than
duplicating all that stuff. 10l to me...
> Fixed locally.
>
Fixed now in github also.
>
>>
>>
>>> + *
>>> + * @param module the AVSequencerModule to be registered
>>> + * @return >= 0 on success, error code otherwise
>>> + *
>>> + * @NOTE This is part of the new sequencer API which is still under construction.
>>>
>>>
>> @note (case is meaningful).
>>
>>
>
> Oooops...fixed locally!
>
Fixed now in github also.
> Please note I can't commit fixes to github (see my eMail) anymore since
> git rebase -i! Please see my mail. So for now, fixed only locally.
>
Fixed. ;-)
Please update your repository pointing to
http://github.com/BastyCDGS/ffmpeg-soc
One question though, should I mail such small patches here also, or is
it enough to mention that they are fixed?
--
Best regards,
:-) Basty/CDGS (-:
More information about the FFmpeg-soc
mailing list