From: Jelmer Vernooij Date: Fri, 27 May 2005 18:36:23 +0000 (+0000) Subject: r7037: Add enough pointers in the header file as well X-Git-Tag: samba-misc-tags/initial-v4-0-unstable~9149 X-Git-Url: http://git.samba.org/?p=samba.git;a=commitdiff_plain;h=fcaa9d495c13af5065dd32f8ed65d4260b3a5539 r7037: Add enough pointers in the header file as well Fix a couple of warnings. --- diff --git a/source/build/pidl/ndr_header.pm b/source/build/pidl/ndr_header.pm index 05c2f1f55c3..05265bbf5bd 100644 --- a/source/build/pidl/ndr_header.pm +++ b/source/build/pidl/ndr_header.pm @@ -60,20 +60,24 @@ sub HeaderElement($) pidl tabs(); HeaderType($element, $element->{TYPE}, ""); pidl " "; - my $pointers = 0; + my $prefix = ""; + my $postfix = ""; foreach my $l (@{$element->{LEVELS}}) { if (($l->{TYPE} eq "POINTER")) { next if ($element->{TYPE} eq "string"); - pidl "*"; - $pointers+=1; - } elsif ($l->{TYPE} eq "ARRAY") { - if (!$pointers and !$l->{IS_FIXED}) { pidl "*"; } - pidl "$element->{NAME}"; - if ($l->{IS_FIXED}) { pidl "[$l->{SIZE_IS}]"; } - last; #FIXME + $prefix .= "*"; + } elsif (($l->{TYPE} eq "ARRAY")) { + my $pl = Ndr::GetPrevLevel($element, $l); + next if ($pl and $pl->{TYPE} eq "POINTER"); + + if ($l->{IS_FIXED}) { + $postfix .= "[$l->{SIZE_IS}]"; + } else { + $prefix .= "*"; + } } elsif ($l->{TYPE} eq "DATA") { - pidl "$element->{NAME}"; + pidl "$prefix$element->{NAME}$postfix"; } } diff --git a/source/librpc/idl/orpc.idl b/source/librpc/idl/orpc.idl index 4b80c5bfe63..1934a3cbed7 100644 --- a/source/librpc/idl/orpc.idl +++ b/source/librpc/idl/orpc.idl @@ -59,7 +59,7 @@ interface ObjectRpcBaseTypes { GUID id; /* Extension identifier. */ uint32 size; /* Extension size. */ - [size_is((size+7)&~7)] uint8 data[]; /* Extension data. */ + [size_is(((size+7)&~7))] uint8 data[]; /* Extension data. */ } ORPC_EXTENT; @@ -68,7 +68,7 @@ interface ObjectRpcBaseTypes { uint32 size; /* Num extents. */ uint32 reserved; /* Must be zero. */ - [size_is((size+1)&~1,), unique] ORPC_EXTENT **extent; /* extents */ + [size_is(((size+1)&~1))] ORPC_EXTENT **extent; /* extents */ } ORPC_EXTENT_ARRAY;