Personally I'm a fan of returning the first result as that just confuses me:

Readability:

```return (dwResult != ERROR_SUCCESS)```

To prevent '=' coding error:

```return (ERROR_SUCCESS != dwResult)```

Decisions decisions!