On Mon, Feb 09, 2015 at 05:21:59AM +0100, Michael Niedermayer wrote:
> On Mon, Feb 09, 2015 at 01:44:43AM +0100, Clément Bœsch wrote:
> > On Fri, Feb 06, 2015 at 10:10:22PM +0100, Michael Niedermayer wrote:
> > > Signed-off-by: Michael Niedermayer <michae...@gmx.at>
> > > ---
> > > ffprobe.c | 3 ++-
> > > 1 file changed, 2 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/ffprobe.c b/ffprobe.c
> > > index d352bb6b..8617c96 100644
> > > --- a/ffprobe.c
> > > +++ b/ffprobe.c
> > > @@ -294,7 +294,8 @@ typedef enum {
> > > WRITER_STRING_VALIDATION_FAIL,
> > > WRITER_STRING_VALIDATION_REPLACE,
> > > WRITER_STRING_VALIDATION_IGNORE,
> > > - WRITER_STRING_VALIDATION_NB
> > > + WRITER_STRING_VALIDATION_NB,
> > > + WRITER_STRING_VALIDATION_FORCE_ENUM_TO_INT_TYPE= 0x76543210,
> > > } StringValidation;
> > >
> >
> > I prefer the int approach you originally proposed. This hack makes me
> > quite uncomfortable, and the idea of having it all over the code base as
> > well.
>
> perfectly fine with me as well, saste i think you preferred above?
> or did i misunderstand ?question timedout will apply original patchset and continue along its lines [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User questions about the command line tools should be sent to the ffmpeg-user ML. And questions about how to use libav* should be sent to the libav-user ML.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
