Re: [PATCH] xenstored: Remove setjmp.h

[Jason Andryuk <jandryuk@xxxxxxxxx>]

On Fri, Apr 18, 2025 at 6:51 PM Julien Grall <julien@xxxxxxx> wrote:
>
> Hi Jason,
>
> On 19/04/2025 05:59, Jason Andryuk wrote:
> > Use of setjmp/longjmp as removed in 2006, but the include remained.

Whoops, this should be "was removed".

> > Remove it now.
> >
> > Fixes: 1bac3b49cd ("Import the current version of talloc...")
>
> The subject truncation is ok however for the commit ID, we are expecting
> 12 characters (see docs/process/sending-patches.pandoc).

Sorry about that.  I used `--pretty=fixes`, but I guess I have it
misconfigured on that machine.

The 12 character ID is:
1bac3b49cdd4

>
> > Signed-off-by: Jason Andryuk <jason.andryuk@xxxxxxx>
>
> With the remark above addressed (can be fixed on commit):
>
> Reviewed-by: Julien Grall <jgrall@xxxxxxxxxx>

Thanks,
Jason