qemu-s390x
[Top][All Lists]
Advanced

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

Re: [PATCH 32/32] Remove qemu-common.h include from most units


From: Markus Armbruster
Subject: Re: [PATCH 32/32] Remove qemu-common.h include from most units
Date: Wed, 06 Apr 2022 12:45:19 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)

First of all: thank you so much for completing the "empty out
qemu-common.h" job!

This is what's left:

    #ifndef QEMU_COMMON_H
    #define QEMU_COMMON_H

    /* Copyright string for -version arguments, About dialogs, etc */
    #define QEMU_COPYRIGHT "Copyright (c) 2003-2022 " \
        "Fabrice Bellard and the QEMU Project developers"

    /* Bug reporting information for --help arguments, About dialogs, etc */
    #define QEMU_HELP_BOTTOM \
        "See <https://qemu.org/contribute/report-a-bug> for how to report 
bugs.\n" \
        "More information on the QEMU project at <https://qemu.org>."

    #endif

Rename the header?  Or replace the macros by variables, and move their
declarations elsewhere?  Not demands; this series is lovely progress as
is.

Ani Sinha <ani@anisinha.ca> writes:

> On Wed, Mar 23, 2022 at 9:32 PM <marcandre.lureau@redhat.com> wrote:
>
>> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>>
>> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
>
>
> Empty patch descriptions are not a good idea. Please explain why this
> removal does not break things and maybe why the header got included
> initiatively.

Something like

    The previous commits moved most of qemu-common.h's contents
    elsewhere.  Most inclusions are now useless.  Remove them.

would do for me.




reply via email to

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