Re: [ros-dev] Error handling

[Jakob Eriksson]

Casper Hornstrup wrote:

>   The control flow isn't obvious since NTCALL hides
> much of the logic. You
>  should rather split the function into smaller functions with sensible
>  names. This will, in many cases, reduce the duplicated error handling 
>  code. 

Indeed. This was my immediate thought too. Maybe something like
this, or maybe not, but you get the idea.


DoNtXXX()
{
    NSTATUS Status;

    Status = NtXXX();

    cleanupXXX();

    return !NT_SUCCESS(Status);
}


DoNtZZZ()
{
    NSTATUS Status;

    Status = NtZZZ();

    cleanupZZZ();

    return !NT_SUCCESS(Status);
}

DoNtYYY()
{
    NSTATUS Status;

    Status = NtYYY();

    cleanupYYY();

    return !NT_SUCCESS(Status);
}


NTSTATUS DoSomething()
{
    return DoNtXXX() || DoNtYYY() || DoNtZZZ();
}