On 1/3/26 08:47, Pierrick Bouvier wrote:
This change has two benefits:
- ensure plugins can't include anything else from QEMU than plugins API
- when compiling a C++ module, solves the header conflict with iostream
   header that includes transitively the wrong ctype.h, which already
   exists in include/qemu.

By Hyrum's law, there was already one usage of other headers with mem
plugin, which has been eliminated in previous commit.

Signed-off-by: Pierrick Bouvier<[email protected]>
---
  docs/devel/tcg-plugins.rst              | 4 ++--
  meson.build                             | 2 +-
  include/{qemu => plugins}/qemu-plugin.h | 3 ---
  include/qemu/plugin.h                   | 2 +-
  plugins/core.c                          | 2 +-
  plugins/meson.build                     | 6 +++---
  scripts/clean-includes                  | 2 +-
  7 files changed, 9 insertions(+), 12 deletions(-)
  rename include/{qemu => plugins}/qemu-plugin.h (99%)

Reviewed-by: Richard Henderson <[email protected]>

r~

Reply via email to