|
From: | Markus Armbruster |
Subject: | Re: [PATCH v4 10/19] migration: Clean up includes |
Date: | Fri, 20 Jan 2023 08:19:04 +0100 |
User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
"Dr. David Alan Gilbert" <dgilbert@redhat.com> writes: > * Markus Armbruster (armbru@redhat.com) wrote: >> "Dr. David Alan Gilbert" <dgilbert@redhat.com> writes: >> >> > * Markus Armbruster (armbru@redhat.com) wrote: >> >> Clean up includes so that osdep.h is included first and headers >> >> which it implies are not included manually. >> > >> > That change doesn't seem to match the message; the patch is removing the >> > osdep.h include. >> >> It's the commit message scripts/clean-includes creates :) >> >> I can throw in another patch to the script so it mentions it also >> removes qemu/osdep.h from headers. > > Oh hmm it would be clearer; What about $GITSUBJ: Clean up includes Clean up includes so that osdep.h is included first in .c and not in .h, and headers which it implies are not included manually. This commit was created with scripts/clean-includes. > but OK then, so > > Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Thanks!
[Prev in Thread] | Current Thread | [Next in Thread] |