[FFmpeg-devel] [PATCH] Other configure version check patches

Stefano Sabatini stefano.sabatini-lala
Sun Oct 19 12:03:07 CEST 2008


On date Saturday 2008-10-18 22:20:08 +0100, M?ns Rullg?rd encoded:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Sat, Oct 18, 2008 at 10:03:48PM +0100, M?ns Rullg?rd wrote:
> >> Diego Biurrun <diego at biurrun.de> writes:
> >> 
> >> > On Sat, Oct 18, 2008 at 08:51:01PM +0100, M?ns Rullg?rd wrote:
[...]
> >> >> > --- configure	(revision 15633)
> >> >> > +++ configure	(working copy)
> >> >> > @@ -1316,6 +1316,10 @@
> >> >> > +        check_cpp_condition w32api.h "(__W32API_MAJOR_VERSION > 3) || (__W32API_MAJOR_VERSION == 3 && __W32API_MINOR_VERSION >= 12)" ||
> >> >> > +            die "w32api must be >= 3.12."
> >> >> 
> >> >> Those lines are frightfully long, but I don't see any better alternative.
> >> >
> >> > For example:
> >> >
> >> >     check_cpp_condition w32api.h "(__W32API_MAJOR_VERSION > 3) || (__W32API_MAJOR_VERSION == 3
> >> >                                 && __W32API_MINOR_VERSION >= 12)" || die "w32api must be >= 3.12."
> >> 
> >> If a line is longer than 80 characters, it doesn't really get worse
> >> until it hits 160.
> >
> > It could be split at the || as well...
> 
> I don't like quoted strings spanning multiple lines either.  I would
> just go with the single line.  The alternatives aren't really any
> easier to read.
> 
> >> BTW, your split would need a backslash or two.
> >
> > No, the quotes take care of this.
> 
> The quotes quote the newline, upon which the preprocessor will choke.

Yes, while long lines are ugly I think in this case they provide the
more robust solution.

In this new patch I just added the missing "ERROR:" in the messages.

Regards.
-- 
FFmpeg = Faboulous and Forgiving Multipurpose Patchable Exxagerate Gangster
-------------- next part --------------
A non-text attachment was scrubbed...
Name: check-mingw-version-00.patch
Type: text/x-diff
Size: 802 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081019/c3cf8269/attachment.patch>



More information about the ffmpeg-devel mailing list