[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-ddrescue] [PATCH 7/7] [clang-tidy] Sort includes according to L

From: Antonio Diaz Diaz
Subject: Re: [Bug-ddrescue] [PATCH 7/7] [clang-tidy] Sort includes according to LLVM rules
Date: Wed, 16 Oct 2019 19:34:53 +0200
User-agent: Mozilla/5.0 (X11; U; Linux i586; en-US; rv: Gecko/20110420 SeaMonkey/2.0.14

Rosen Penev wrote:
Found with llvm-include-order

Rules are, C++ headers, then C, then private.

Found what? Ddrescue already includes headers in the above order. The only thing this patch seems to do is to add a blank line between C++ and C headers and randomly reorder some headers inside each group. Nothing useful, much less a bug.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]