Re: [PATCH 3/4] include: Define IoSkipCurrentIrpStackLocation

[Alexandre Julliard]

Sebastian Lackner  writes:
> I assume you are looking at an old version then, and Microsoft fixed
> this mistake in the meantime. Windows 10 header files contain for example:
>
> --- snip ---
> FORCEINLINE
> VOID
> IoSkipCurrentIrpStackLocation (
>     _Inout_ PIRP Irp
> )
> [...]
> {
>     NT_ASSERT(Irp->CurrentLocation <= Irp->StackCount);
>     Irp->CurrentLocation++;
>     Irp->Tail.Overlay.CurrentStackLocation++;
> }
> --- snip ---
>
> Nevertheless, I think both "{...}" and using an inline function is fine
> in this case.

It should be either an inline function (preferred) or a "do {} while(0)"
block. A "{...}" block will cause syntax errors in some cases.