getting rid of utf8string2

Stefan (metze) Metzmacher metze at samba.org
Wed Mar 24 01:24:34 MDT 2010


Hi Simo,


> diff --git a/librpc/idl/idl_types.h b/librpc/idl/idl_types.h
> index e8b3da5..d763a8b 100644
> --- a/librpc/idl/idl_types.h
> +++ b/librpc/idl/idl_types.h
> @@ -48,9 +48,10 @@
>  #define utf8string	[flag(STR_UTF8|STR_NULLTERM)] string
>  
>  /*
> -  an utf8 string prefixed with [size], 32 bits
> +  an utf8 string prefixed with [length], 32 bits

This should be [size] when you use STR_SIZE4...

> +  not null terminated
>  */
> -#define utf8string2	[flag(STR_UTF8|STR_SIZE4)] string
> +#define utf8string2	[flag(STR_SIZE4|STR_UTF8|STR_NOTERM)] string

I'd like to get rid of this as you can express the same with valid NDR.

Something like:

[value(strlen(dns_name))] uint32 dns_name_size;
[charset(UTF8)] uint8 dns_name[dns_name_size];

should do the same.

Can you send me a file I can pass to ndrdump to test it?

metze

PS: It would be nice if you wouldn't introduce leading whitespaces in
the idl file...:-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 260 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-cvs/attachments/20100324/778ab03f/attachment.pgp>


More information about the samba-cvs mailing list