On Wed, 4 Sep 2024 17:58:49 +0800 Kefeng Wang <wangkefeng.wang@xxxxxxxxxx> wrote: > Some includes are not need anymore, remove them. > Please define "not needed". Does migrate.c simply not refer to anything which these headers provide? Or does migrate.c already pick up these headers via nested includes? If the latter then that can be a problem - build breakage under rare configs, or as code generally evolves. Not a *big* problem - we detect and fix such things quickly, but I do think general good practice is to directly include whatever the .c file requires.
