James Hawkins wrote: 
On Thu, Oct 9, 2008 at 4:14 PM, Andrey Turkin <andrey.turkin@gmail.com> wrote:
  
[now with tests, fixed and reformatted as per James' suggestions]

Implement CredReadDomainCredentialsA and CredReadDomainCredentialsW
stubs and few tests for them.  Required for MSN Messenger 7.0
---
 dlls/advapi32/advapi32.spec |    4 +-
 dlls/advapi32/cred.c        |  160 +++++++++++++++++++++++++++++++++++++++++++
 dlls/advapi32/tests/cred.c  |   45 ++++++++++++
 include/wincred.h           |   39 +++++++++++
 4 files changed, 246 insertions(+), 2 deletions(-)

    

+    *Size = 0;

+    *Credentials = NULL;

+    if (!TargetInformation)

What if Size or Credentials is NULL?  You should add a test for that
and handle it appropriately.  If the tests crash, you should comment
out the tests and note that there is no handling in Windows.

Why? Windows does the same thing, tests would crash. Do you really think somebody need to know this? Is there any application that depends on this behavior so these assignments must be commented to protect them from change? I can't see any value in commented out test or sort-of-meaningless code comment.