r1225 - /unstable/ffmpeg/debian/patches/300_c++_compliant_headers.diff

[siretart at users.alioth.debian.org]

Author: siretart
Date: Tue Jun  3 14:41:03 2008
New Revision: 1225

URL: http://svn.debian.org/wsvn/pkg-multimedia/?sc=1&rev=1225
Log:
make note about discussion on patch 300_c++_compliant_headers.diff

Modified:
    unstable/ffmpeg/debian/patches/300_c++_compliant_headers.diff

Modified: unstable/ffmpeg/debian/patches/300_c++_compliant_headers.diff
URL: http://svn.debian.org/wsvn/pkg-multimedia/unstable/ffmpeg/debian/patches/300_c%2B%2B_compliant_headers.diff?rev=1225&op=diff
==============================================================================
--- unstable/ffmpeg/debian/patches/300_c++_compliant_headers.diff (original)
+++ unstable/ffmpeg/debian/patches/300_c++_compliant_headers.diff Tue Jun  3 14:41:03 2008
@@ -1,3 +1,13 @@
+The following patch has been discussed on #ffmpeg-devel on 2008-06-03:
+
+16:30:04 <@DonDiego> 300_c++_compliant_headers.diff
+16:30:13 <@DonDiego> siretart: this patch is wrong for two reasons
+16:31:21 <@DonDiego> 1) the header is not #included everywhere that INT64_C is used
+16:31:44 <@DonDiego> 2) if this is required for C++, then C++ programs should add the definition themselves
+16:31:53 <@DonDiego> it has no place in a C-only library
+
+Todo: Check if any package actually relies on that.
+
 Index: ffmpeg.20080206/libavcodec/avcodec.h
 ===================================================================
 --- ffmpeg.20080206.orig/libavcodec/avcodec.h	2008-03-15 22:59:25.000000000 +0100