[ros-dev] Re: [ros-svn] [gdalsnes] 18113: -reorder InsertXscendingOrder macro argument order and update uses

[Nathan Woods]

Gunnar Dalsnes wrote:

> In win32k i use macros for flow control in the NtUser syscalls 
> (functionally like a try/finally block) where its very important that 
> some cleanup (release a lock) is always done:
>
> BOOL NtFunc()
> {
>   DECLARE_RETURN(BOOL);
>
>   Lock();
>   if (Stuff) RETURN(FALSE);
>   ....
>   RETURN(TRUE);
>
> CLEANUP:
>   Unlock(Stuff);
>   DPRINT1("NtFunc returned %i\n", _ret_);
>   END_CLEANUP;
> }

Patterns like this can be done without macros.  I've always been partial 
to the following style:

BOOL NtFunc()
{
  BOOL bResult;
  void *pPointer = NULL;

  Lock();

  if (Stuff)
  {
    bResult = FALSE;
    goto cleanup;
  }
  ....

  bResult = TRUE;

cleanup:
  if (pPointer)
    free(pPointer);
  Unlock(stuff);
  DPRINT1("NtFunc returned %i\n", bResult);
  return bResult;
}

I know that many find goto's to be a crime against humanity, but I've 
always favored them for this sort of pattern.

-N