Remove NTSTATUS from NDR Layer

[Stefan (metze) Metzmacher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Jelmer,

>> It passes make test and I think I have reviewed the diffs
>> carefully, but please check careful too.
> Looks good overall!

How should I commit this?
3 patches pidl/, librpc/ndr/ + all the callers?

> A general thought: ndr_map_error2ntstatus() should support more
> ndr_err_code values. That was already the case before your patch though,
> so I think it's ok if we fix that later.

yes

> Some minor issues:
> 
> I would prefer a "const char *ndr_errstring(enum ndr_err_code);"
> function over converting to NTSTATUS and then calling ndr_errstr() on
> that each time.

Yes, but can be the next patch.

> The name RAP_GOTO() is unclear imho - something like RAP_CHECK() or
> RAP_CHECK_GOTO() would be clearer. 
> 
> NDR_GOTO() and NDR_RETURN() are present in more than one C file - I'd
> rather see them in torture/ndr/ndr.h.

The rap code needs sharing between torture/rap/ and ntvfs/ipc/
in general, so I just leave it to have the same code twice
as I don't want to think about the real solution as it would be
unrelated to my scope.

metze
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)
Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org

iD8DBQFHMBQmm70gjA5TCD8RAmhCAJ9cJNS2AYrJqSioZUQLIrrMpJ0ycQCeML0y
7x8qoHyYTWxLCwDZwYX8Bgo=
=XfMb
-----END PGP SIGNATURE-----