[FFmpeg-devel] [PATCH] Reduce MAKE_ACCESSORS code duplication via a new header.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Fri Aug 9 06:34:19 CEST 2013


On 08.08.2013, at 23:55, Paul B Mahol <onemda at gmail.com> wrote:
> 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.

Sorry, I don't know what I was thinking. I'll propose moving it when I have time, if nobody beats me to it.


More information about the ffmpeg-devel mailing list