[PATCH 3/4] include: Define IoSkipCurrentIrpStackLocation

Alexandre Julliard julliard at winehq.org
Thu Oct 13 02:09:23 CDT 2016


Sebastian Lackner <sebastian at fds-team.de> 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.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list