Re: [PATCH] doc: fix some typos
- From
- Martin Ågren <martin.agren@gmail.com>
- Date
- Jan 2, 2021, 21:53 UTC
- Message-ID
- <CAN0heSojPyr=g5PBVmT4VTfxxKxzYcFXJ1jynM1tGuZSyQMykg@mail.gmail.com>
- In-Reply-To
- <pull.831.git.1609609214040.gitgitgadget@gmail.com>
On Sat, 2 Jan 2021 at 18:43, Thomas Ackermann via GitGitGadget <gitgitgadget@gmail.com> wrote:
[Snip several typo fixes in Documentation/, all of which I agree with.]
> diff --git a/contrib/buildsystems/CMakeLists.txt b/contrib/buildsystems/CMakeLists.txt > index c151dd7257f..8f77baa678f 100644 > --- a/contrib/buildsystems/CMakeLists.txt > +++ b/contrib/buildsystems/CMakeLists.txt
Hmm... This does not match the "doc:" prefix of the patch.
Show 7 quoted lines
> @@ -442,7 +442,7 @@ endif()
> check_c_source_compiles("
> #include <regex.h>
> #ifndef REG_STARTEND
> -#error oops we dont have it
> +#error oops we don't have it
> #endifI don't think this is correct. This omission of the single quote somehow looks like it's done on purpose. I don't build using this system, but I tried making some silly code like this in another file, which I actually do use:
#if 0 #error might or mightn't work #endif
My compiler refused to compile: "error: missing terminating ' character [-Werror]". So I fear this change to CMakeLists.txt would make us needlessly fail to autodetect whatever it is we're looking for here.
If this change to CMakeLists.txt is dropped, this patch looks good to me.
Martin