[PATCH 1/2] clang-format: Make Qt includes matching case sensitive
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Mon Jun 10 20:56:40 CEST 2024
Hi Milan,
Thank you for the patch.
On Mon, Jun 10, 2024 at 08:08:49PM +0200, Milan Zamazal wrote:
> This fixes the problem that includes like
>
> #include <queue>
>
> are put near the end.
>
> Signed-off-by: Milan Zamazal <mzamazal at redhat.com>
> ---
> .clang-format | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.clang-format b/.clang-format
> index cac7029f..b48d4e1e 100644
> --- a/.clang-format
> +++ b/.clang-format
> @@ -75,6 +75,7 @@ IncludeCategories:
> Priority: 9
> # Qt includes (match before C++ standard library)
> - Regex: '<Q([A-Za-z0-9\-_])+>'
> + CaseSensitive: true
This option is available in clang-format 12 and later. There are no
distribution versions we specifically care about that ship a too old
clang-formation version (Debian buster ships 11 by default but has a
clang-format-13 package), so this is fine. We should however update the
comment at the beginning of the file to indicate this. I can handle it
when applying this patch.
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
By the way, see commit d1cdaeb6f078677d5bf46cb596102bfe8da070b9 :-)
> Priority: 9
> # Headers in <> with an extension. (+system libraries)
> - Regex: '<([A-Za-z0-9\-_])+\.h>'
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list