[vlc-devel] [PATCH] demux: mkv: fix compilation by gcc492 on OS/2

[KO Myung-Hun]

Jean-Baptiste Kempf wrote:
> On 04 May, KO Myung-Hun wrote :
>> -----
>>   CXX      demux/mkv/libmkv_plugin_la-virtual_segment.lo
>> In file included from ../include/vlc_common.h:38:0,
>>                  from demux/mkv/mkv.hpp:45,
>>                  from demux/mkv/demux.hpp:28,
>>                  from demux/mkv/virtual_segment.cpp:27:
>> demux/mkv/demux.hpp: In constructor 'demux_sys_t::demux_sys_t(demux_t&)':
>> ../include/vlc_config.h:43:33: error: 'INT64_C' was not declared in this scope
>>  #define VLC_TS_INVALID INT64_C(0)
>>                                  ^
>> demux/mkv/demux.hpp:334:16: note: in expansion of macro 'VLC_TS_INVALID'
>>          ,i_pts(VLC_TS_INVALID)
>>                 ^
>> demux/mkv/virtual_segment.cpp: In member function 'bool virtual_segment_c::UpdateCurrentToChapter(demux_t&)':
>> ../include/vlc_config.h:43:33: error: 'INT64_C' was not declared in this scope
>>  #define VLC_TS_INVALID INT64_C(0)
>>                                  ^
>> demux/mkv/virtual_segment.cpp:407:23: note: in expansion of macro 'VLC_TS_INVALID'
>>      if ( sys.i_pts != VLC_TS_INVALID )
>>                        ^
>> ../include/vlc_config.h:44:27: error: 'INT64_C' was not declared in this scope
>>  #define VLC_TS_0 INT64_C(1)
>>                            ^
>> demux/mkv/virtual_segment.cpp:430:76: note: in expansion of macro 'VLC_TS_0'
>>                  sys.i_start_pts = p_cur_chapter->i_mk_virtual_start_time + VLC_TS_0;
>>                                                                             ^
>> make.exe[4]: *** [demux/mkv/libmkv_plugin_la-virtual_segment.lo] Error 1
>> -----
>> ---
>>  modules/demux/mkv/virtual_segment.cpp | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/modules/demux/mkv/virtual_segment.cpp b/modules/demux/mkv/virtual_segment.cpp
>> index c7852d6..5eca79d 100644
>> --- a/modules/demux/mkv/virtual_segment.cpp
>> +++ b/modules/demux/mkv/virtual_segment.cpp
>> @@ -22,7 +22,6 @@
>>   * along with this program; if not, write to the Free Software Foundation,
>>   * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
>>   *****************************************************************************/
>> -#include <vector>
> 
> This is, of course, incorrect.
> 

vector is included by mkv.hpp included by demux.hpp. Nevertheless vector
is needed specifically ?

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr