llcppcfg: sort includes

[tsingbx]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.01%. Comparing base (81acd8d) to head (2e74219).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #121   +/-   ##
=======================================
  Coverage   98.01%   98.01%           
=======================================
  Files          15       15           
  Lines        1864     1864           
=======================================
  Hits         1827     1827           
  Misses         26       26           
  Partials       11       11           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[luoliwoshang]

这个实现看起来是通过头文件的依赖数量来决定其生成的include顺序，按照依赖数量排序,并不能反映适合的文件处理顺序。

考虑以下场景
main.h

#include "common_type.h"
#include a.h

common_type.h

typedef struct Foo {
    int a;
} Foo;

a.h

#include "a_common.h"
#include  "a_type.h"
#include  "a_compact.h"
Foo *fn();

按照这个由 头文件的依赖数量 的逻辑来输出的include顺序会是 a.h(3) , main.h(2), common_type.h(0),但这个处理顺序一定是非预期的 ，a.h 这里使用Foo时，还没定义Foo类型。

[tsingbx]

无论怎么处理，都必须要有这个，同级头文件，根据什么来判断？

[luoliwoshang]

无论怎么处理，都必须要有这个，同级头文件，根据什么来判断？

• 同级别的头文件通过其引用include关系来处理其顺序。

• 当前根据依赖数量的逻辑来给出include处理顺序并不合适，该逻辑对于我上面提出的用例已经不能正确表达了。