From a9531c0e23b622cb96c6c4a61a0df43d12a3cee8 Mon Sep 17 00:00:00 2001
From: Stefan Metzmacher <metze@samba.org>
Date: Fri, 17 Aug 2007 12:55:47 +0000
Subject: r24517: move skipping pointer before an array logic into the pointer
 loop as we do in other places

metze
(This used to be commit ee92d47b538e2f92e4c342893f04c598311e1021)
---
 source4/pidl/lib/Parse/Pidl/Samba4/Header.pm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

(limited to 'source4/pidl')

diff --git a/source4/pidl/lib/Parse/Pidl/Samba4/Header.pm b/source4/pidl/lib/Parse/Pidl/Samba4/Header.pm
index f03f7c0c43..9d8b521a49 100644
--- a/source4/pidl/lib/Parse/Pidl/Samba4/Header.pm
+++ b/source4/pidl/lib/Parse/Pidl/Samba4/Header.pm
@@ -10,7 +10,7 @@ use strict;
 use Parse::Pidl qw(fatal);
 use Parse::Pidl::Typelist qw(mapTypeName scalar_is_reference);
 use Parse::Pidl::Util qw(has_property is_constant);
-use Parse::Pidl::NDR qw(GetPrevLevel);
+use Parse::Pidl::NDR qw(GetNextLevel);
 use Parse::Pidl::Samba4 qw(is_intree);
 
 use vars qw($VERSION);
@@ -68,6 +68,10 @@ sub HeaderElement($)
 		my $numstar = 0;
 		foreach my $l (@{$element->{LEVELS}}) {
 			next unless ($l->{TYPE} eq "POINTER");
+
+			my $nl = GetNextLevel($element, $l);
+			next if (defined($nl) and $nl->{TYPE} eq "ARRAY");
+
 			$numstar++;
 		}
 		if ($numstar >= 1) {
@@ -77,8 +81,6 @@ sub HeaderElement($)
 			next unless ($l->{TYPE} eq "ARRAY");
 			next if ($l->{IS_FIXED}) and
 				not has_property($element, "charset");
-			my $pl = GetPrevLevel($element, $l);
-			next if (defined($pl) and $pl->{TYPE} eq "POINTER");
 			$numstar++;
 		}
 		pidl "*" foreach (1..$numstar);
-- 
cgit