[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2 0/6] Remove dependency on __LINE__
>>> On 08.03.17 at 18:46, <ross.lagerwall@xxxxxxxxxx> wrote: > Sorry for the long delay since the first version of this series > (previously called "Make building xSplice patches easier"). Here is a > set of changes that remove the use of __LINE__ when building with NDEBUG > and LivePatch enabled. Tested to boot on x86. Compile-tested on arm. What I'm missing here is an evaluation of possible alternatives, as converting from __LINE__ to code addresses implies an extra step when wanting to look up the place where a problem occurred, which I generally consider undesirable. For example, I had briefly discussed with Andrew the addition of #line directives, but iirc he had indicated some issue with that. Furthermore there is the option (at least for some of the more simple patches) of squeezing more than one statement on a single line, perhaps just for patch creation. And I guess one can think of more. Jan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |