svn commit: samba r22640 - in branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4: .

Jelmer Vernooij jelmer at samba.org
Wed May 2 19:08:51 GMT 2007


Hi Metze,

metze at samba.org wrote:
> Author: metze
> Date: 2007-05-02 18:00:02 +0000 (Wed, 02 May 2007)
> New Revision: 22640
> 
> WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=22640
> 
> Log:
> - generate nicer output
> - fix compiler warning about unused ';'
> 
> metze
> Modified:
>    branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/Header.pm
> 
> 
> Changeset:
> Modified: branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/Header.pm
> ===================================================================
> --- branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/Header.pm	2007-05-02 16:10:23 UTC (rev 22639)
> +++ branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/Header.pm	2007-05-02 18:00:02 UTC (rev 22640)
> @@ -43,7 +43,7 @@
>  	}
>  
>  	if ($ret) {
> -		pidl "/* [" . substr($ret, 0, -1) . "] */";
> +		pidl " /* [" . substr($ret, 0, -1) . "] */";
>  	}
>  }
>  
> @@ -108,10 +108,12 @@
>  	    pidl tabs()."char _empty_;\n";
>      }
>      $tab_depth--;
> -    pidl tabs()."}";
> +
> +    pidl "};";

I'm pretty sure this breaks support for nested structures / unions.

Cheers,

Jelmer


More information about the samba-cvs mailing list