please don't apply RPC Merge D_PL1

[Greg Turner]

On Sunday 13 October 2002 08:20 pm, Greg Turner wrote:
> On Sunday 13 October 2002 06:41 pm, Ove Kaaven wrote:
> > On Sun, 13 Oct 2002, Greg Turner wrote:
> > > LONG_PTR RPCRT4_NdrClientCall2(PMIDL_STUB_DESC pStubDesc,
> > > PFORMAT_STRING pFormat, va_list args) {
> > >
> > >   FIXME("(pStubDec == ^%p,pFormat = \"%s\",...): stub\n", pStubDesc,
> > > pFormat);
> > >
> > >   PRPC_CLIENT_INTERFACE rpc_cli_if =
> > > (PRPC_CLIENT_INTERFACE)(pStubDesc->RpcInterfaceInformation);
> >
> > (hmm, does this even compile? I didn't think C allowed variable
> > declarations after statements)
>
> This is a c++ism that gcc allows; it already is used quite a bit in wine,
> although in this case there is no good reason for it and it should probably
> go.
>
> > The pStubDesc structure should have been initialized (including setting
> > that pointer, I think) by the NdrClientInitialize or
> > NdrClientInitializeNew (or for DCOM, NdrProxyInitialize, but perhaps
> > NdrProxyInitialize should call down to NdrClientInitialize/New anyway)
> > functions. My patches didn't implement those functions, since I was
> > mostly concerned with DCOM. Is this a real RPC app that works on Windows
> > you're working on?
>
> It's "Samples\netds\rpc\Hello" from the latest platform SDK, compiled
> native.  It does seem to work in real Windows.  Binaries are enclosed.
> I will look into some of the functions you mention after Sopranos :)
>
> Thanks, Ove,

did these come through for everybody?  I got a message from an 
Exchange bot about virus blocking.  Here they are again in a format that
an MS bot is unlikely to censor.

-- 
gmt

"It has been well said that really up-to-date liberals
do not care what people do, as long as it is compulsory."

-George F. Will
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hello.tbz2
Type: application/x-bzip2
Size: 3323 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20021013/56474057/hello.bin