Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-11 Thread Max TenEyck Woodbury
On 11/11/2012 03:05 PM, Michael Stefaniuc wrote: > On 11/11/2012 07:12 AM, Max TenEyck Woodbury wrote: >> On 11/11/2012 01:01 AM, Nikolay Sivov wrote: >>> On 11/11/2012 05:00, Max TenEyck Woodbury wrote: I mentioned this a few days ago. It would have helped if you had raised this point t

Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-11 Thread Michael Stefaniuc
On 11/11/2012 07:12 AM, Max TenEyck Woodbury wrote: > On 11/11/2012 01:01 AM, Nikolay Sivov wrote: >> On 11/11/2012 05:00, Max TenEyck Woodbury wrote: >>> I mentioned this a few days ago. It would have helped if you had >>> raised this point then. >>> >>> As it stands, it is simply a way to adding

Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-11 Thread Michael Stefaniuc
On 11/11/2012 03:00 AM, Max TenEyck Woodbury wrote: > I mentioned this a few days ago. It would have helped if you had > raised this point then. I tried, but your email made no sense. You didn't even mention that you're talking about COM interfaces. > As it stands, it is simply a way to adding da

Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-10 Thread Max TenEyck Woodbury
On 11/11/2012 01:01 AM, Nikolay Sivov wrote: > On 11/11/2012 05:00, Max TenEyck Woodbury wrote: >> I mentioned this a few days ago. It would have helped if you had >> raised this point then. >> >> As it stands, it is simply a way to adding data members to an aggregate >> with an interface. > > Dat

Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-10 Thread Nikolay Sivov
On 11/11/2012 05:00, Max TenEyck Woodbury wrote: I mentioned this a few days ago. It would have helped if you had raised this point then. As it stands, it is simply a way to adding data members to an aggregate with an interface. Data members to an aggregate? What are you talking about and what

Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-10 Thread Max TenEyck Woodbury
I mentioned this a few days ago. It would have helped if you had raised this point then. As it stands, it is simply a way to adding data members to an aggregate with an interface. In that sense it is not an addition to the interface since the Vtbl pointer remains exactly as before. The new info

Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-10 Thread Michael Stefaniuc
You can't do that as the COM standard specifies also the ABI. A COM interface is a pointer to a virtual table; it is *not* a struct. That's just an implementation detail in C. On 11/10/2012 08:48 PM, m...@mtew.isa-geek.net wrote: > From: Max TenEyck Woodbury > > --- > tools/widl/header.c | 14

Re: [PATCH] tools/widl/header.c: Add a way to declare interface data members.

2012-11-10 Thread Nikolay Sivov
On 11/10/2012 22:48, m...@mtew.isa-geek.net wrote: From: Max TenEyck Woodbury --- tools/widl/header.c | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/tools/widl/header.c b/tools/widl/header.c index 2f275c7..965dcbc 100644 --- a/tools/widl/header.c +++ b