[FFmpeg-devel] [PATCH] Reduce MAKE_ACCESSORS code duplication via a new header.
Paul B Mahol
onemda at gmail.com
Thu Aug 8 23:55:02 CEST 2013
On 8/8/13, James Almer <jamrial at gmail.com> wrote:
> On 08/08/13 3:02 PM, Reimar Doeffinger wrote:
>> ---
>> libavcodec/utils.c | 5 +----
>> libavformat/utils.c | 5 +----
>> libavutil/accessors.h | 3 +++
>> libavutil/frame.c | 4 +---
>> 4 files changed, 6 insertions(+), 11 deletions(-)
>> create mode 100644 libavutil/accessors.h
>
> A bit late to the party, but wouldn't it make more sense to add this
> to libavutil/internal.h or some other existing general usage header?
>
> In any case, you should add a copyright header to accessors.h
But than patch would be much bigger.
Switch to one which much less lines.
More information about the ffmpeg-devel
mailing list