[FFmpeg-devel] [PATCH] Makefile: delete unneeded escape
Michael Niedermayer
michaelni at gmx.at
Sun Feb 17 10:50:32 EET 2019
On Sun, Feb 17, 2019 at 09:44:52AM +0100, Reto Kromer wrote:
> Best regards, Reto
> Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> ed5a95e331e302563d0859bd3b514d88376696e6 0001-Makefile-delete-unneeded-escape.patch
> From 764d9459caecc259952984258ae19ed0906c1176 Mon Sep 17 00:00:00 2001
> From: Reto Kromer <lists at reto.ch>
> Date: Sun, 17 Feb 2019 09:40:01 +0100
> Subject: [PATCH] Makefile: delete unneeded escape
>
> ---
> Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index 59b26c65c9..08b3cfa9b1 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -59,7 +59,7 @@ CONFIGURABLE_COMPONENTS = \
> $(wildcard $(FFLIBS:%=$(SRC_PATH)/lib%/all*.c)) \
> $(SRC_PATH)/libavcodec/bitstream_filters.c \
> $(SRC_PATH)/libavcodec/parsers.c \
> - $(SRC_PATH)/libavformat/protocols.c \
> + $(SRC_PATH)/libavformat/protocols.c
>
if someone adds a line after this he would have to change the previous line
(adding the \ back)
that would make such a patch less tidy (2 lines changed instead of 1)
i suspect that was the reason why all lines have a \
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190217/877d2330/attachment.sig>
More information about the ffmpeg-devel
mailing list