[FFmpeg-cvslog] r15002 - in trunk/libavformat: internal.h sdp.c utils.c

[Ronald S. Bultje]

Hi Stefano,

On Fri, Aug 29, 2008 at 10:51 PM, Stefano Sabatini
<stefano.sabatini-lala at poste.it> wrote:
> On date Friday 2008-08-29 19:51:24 -0400, Ronald S. Bultje wrote:
>> On Fri, Aug 29, 2008 at 7:29 PM, Stefano Sabatini
>> <stefano.sabatini-lala at poste.it> wrote:
>> @@ -3075,4 +3075,4 @@
>>  #define AVERROR_NOENT       AVERROR(ENOENT)  /**< No such file or directory. */
>>  #define AVERROR_PATCHWELCOME    -MKTAG('P','A','W','E') /**< Not yet
>> implemented in FFmpeg. Patches welcome. */
>>
>> -#endif /* FFMPEG_AVCODEC_H */
>> +#endif /* AVCODEC_AVCODEC_H */
>>
>> That's kind of ugly?
>
> No I think it's pretty regular and simple -> nice, cannot see other
> rules which are both straightforward and don't lead to possible name
> conflicts.

LIBAVCODEC_AVCODEC_H? Double AVCODEC is ugly. Anyway, minor, rest of
the work is nice. :-). I guess I was just wondering if others agreed.

Ronald