Opened 13 years ago

Last modified 13 years ago

#1850 closed defect (fixed)

[PATCH] demuxer.c improperly includes libass/ass.h

Reported by: gwirth79@… Owned by: reimar
Priority: normal Component: demuxer
Version: HEAD Severity: normal
Keywords: Cc: diego@…
Blocked By: Blocking:
Reproduced by developer: no Analyzed by developer: no

Description

The file libmpdemux/demuxer.c improperly includes libass/ass.h

The decision to include libass/ass.h or <ass/ass.h> is done in sub/ass_mp.h based on whether CONFIG_ASS_INTERNAL is defined or not.

Patch based on latest code checked out from Subversion and generated with "svn diff"

Gus

Index: demuxer.c
===================================================================
--- demuxer.c (revision 32781)
+++ demuxer.c (working copy)
@@ -47,7 +47,6 @@

#include "libmpcodecs/vd_ffmpeg.h"


#ifdef CONFIG_ASS

-#include "libass/ass.h"

#include "sub/ass_mp.h"
#endif

Attachments (1)

demuxer.patch​ (280 bytes ) - added by gwirth79@… 13 years ago.
patch to fix demuxer.c include

Download all attachments as: .zip

Change History (2)

by gwirth79@…, 13 years ago

Attachment: demuxer.patch​ added

patch to fix demuxer.c include

comment:1 by diego@…, 13 years ago

op_sys: Linux → All
Resolution: → fixed
Status: new → closed, diego@biurrun.de

Thanks for the patch, will apply in a moment.

Note: See TracTickets for help on using tickets.