[Top][All Lists]

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

Re: [Qemu-devel] [PATCH v2 02/47] trace: switch io/ directory to modular

From: Lluís Vilanova
Subject: Re: [Qemu-devel] [PATCH v2 02/47] trace: switch io/ directory to modular trace.h file
Date: Thu, 12 Jan 2017 15:29:22 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Paolo Bonzini writes:

> On 12/01/2017 02:02, Lluís Vilanova wrote:
>> Paolo Bonzini writes:
>> [...]
>>> A weird idea: what about doing
>>> -DGENERATED_TRACERS_H=\"hw/scsi/generated-tracers.h\"
>>> and then having
>>> #include GENERATED_TRACE_H
>>> #endif
>>> in include/trace.h?
>>> Then you can use full include path for special cases such as
>>> include/hw/xen/xen_common.h, but the common case is handled directly
>>> with just
>>> #include "trace.h"
>>> which refers to $(srcdir)/include/trace.h? (Take the above with a grain
>>> of salt because I haven't reviewed the patches closely).
>> Feels like too much black magic to me for the benefit of a bit less typing.

> It's not less typing, it's about consistency.   Includes from the
> current directory are currently included with no path.

Ah, right. My subconscious made me ignore that, since I already said it's
cleaner to me to use the "full" path. Sorry about that :)


reply via email to

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