Re: _variant_t Question - COM Discussion Boards

Ed Gadziemski8-Dec-06 18:29

Ed Gadziemski

8-Dec-06 18:29

Try explicitly casting your object to an IDispatch pointer, for example (IDispatch *)pICustom. That should eliminate falling through to bool.

Gupta Suraj10-Dec-06 17:44

Gupta Suraj

10-Dec-06 17:44

Thanks...It worked...

Suraj Gupta

prasad_som10-Dec-06 21:29

10-Dec-06 21:29

Though, you had a solution to your problem;

Gupta Suraj wrote:
bool operator of _com_ptr template class is called.

Which operator bool you are talking about ? In fact, it should return you custom interface type.
Its implementated like this,

operator T*()   {
    return p;

So I feel, it should return interface pointer.
Can you show how you have declared this interface ?

Gupta Suraj wrote:
We don't have a overloaded constructor of _variant_t class which takes smart pointer as its parameter.

No, CComPtr, will be called for operator T*. Which returns appropriate interface pointer.
i.e. consider this declaration,

CComPtr<IMyInterface> pMy;
_variant_t f(pMy);

Here,in second statement pMy->Operator T* will be get called first. Which returns IMyInterface*.
So, it will set for VT_IDISPATCH automatically.
You dont even need to cast it.

Prasad
Notifier using ATL | Operator new[],delete[][^]

Gupta Suraj11-Dec-06 18:28

Gupta Suraj

11-Dec-06 18:28

Hi Prasad,

Here is code snippet of my application..

// Smart pointer(SampleObjects::ISampleSetDualPtr) typedef declarations
_COM_SMARTPTR_TYPEDEF(ISampleSetDual, __uuidof(ISampleSetDual));
// Smart pointer typedef declarations

SampleObjects::ISampleSetDualPtr pMutipleValues = SampleObjects::ISampleSetDualPtr )propertyPtr->Value;

_variant_t vtMultiValue(pMutipleValues);//This statement first causes bool operator of template class _com_ptr_t being called and then overloaded constructor of _variant_t class being called which takes bool as input parameter.

template<typename _iiid=""> class _com_ptr_t
{
......
......
......
operator bool() const throw()
{
return m_pInterface != NULL;
}
}

Please let me know your input on this.

Thanks,
Suraj

Suraj Gupta

prasad_som11-Dec-06 19:22

11-Dec-06 19:22

Now I got your point.
I was in assumption, you are using CComPtr, where as you are using _com_ptr_t . Both are having different implementations.

Gupta Suraj wrote:
_variant_t vtMultiValue(pMutipleValues);//This statement first causes bool operator of template class _com_ptr_t being called and then overloaded constructor of _variant_t class being called which takes bool as input parameter

Here, compiler looks for mathcing constructor of _variant_t. It doesn't find one with given smart pointer. _com_ptr_t is overloaded for operator bool(). Hence, there is match betn constructor with bool parameter of _variant_t and passed smart pointer variable.

Type casting as said in one of previous replies is one solution.
Or you can Use

_variant_t vtMultiValue(pMutipleValues.GetInterfacePtr());//

to resolve ambiguity .

Prasad
Notifier using ATL | Operator new[],delete[][^]

Gupta Suraj11-Dec-06 19:36

Gupta Suraj

11-Dec-06 19:36

Thank you...

Solution you suggested works.

I thought another solution that works in this case is

_variant_t vtMultiValue(pMutipleValues, true);

You have being a great help.
Suraj

Suraj Gupta

prasad_som11-Dec-06 19:51

Automation, cross thread problem

11-Dec-06 19:51

Gupta Suraj wrote:
_variant_t vtMultiValue(pMutipleValues, true);

Yes, that's becuase, as in previous case compiler was picking _variant_t( bool bSrc ) throw( ) version of constructor as match.

But, in this case that wont happen because of extra parameter provided(That resolved ambuguity in one way).
And compiler matches this construcor _variant_t( IDispatch* pDispSrc, bool fAddRef = true ) throw( ); To do that match, first parameter is typecasted with IDispatch*.

Prasad
Notifier using ATL | Operator new[],delete[][^]

anderslundsgard6-Dec-06 21:36

anderslundsgard

6-Dec-06 21:36

I’m automating MS Project and when I listen on an event from MS Project all works fine but when I’m trying to fill a TreeView I got the following exception:

Action being performed on this control is being called from the wrong thread. Marshal to the correct thread using Control.Invoke or Control.BeginInvoke to perform this action.

What have I missed?

_____________________________

...and justice for all

APe

«_Superman_»6-Dec-06 22:50

«_Superman_»

6-Dec-06 22:50

I'm guessing you got the TreeView control from one thread and are filling from another.

« Superman »

Roger Stoltz6-Dec-06 22:50

Roger Stoltz

6-Dec-06 22:50

Interfaces that cross apartment boundaries has to be marshaled.
No surprises here, it's according to the COM spec.

When you spawn a thread that should use COM stuff, you create another apartment and you have to initalize COM for that apartment with a call to ::CoInitalize() or ::CoInitializeEx(). If you want to use a COM interface that you got in another thread you have to marshal that interface to the new apartment.
There are several ways of doing this, but I think in your case the easiest way would be to use ::CoMarshalInterThreadInterfaceInStream() and ::CoGetInterfaceAndReleaseStream().
Have a look at MSDN here[^].

--
Roger

"It's supposed to be hard, otherwise anybody could do it!" - selfquote

"No one remembers a coward!" - Jan Elfström 1998
"...but everyone remembers an idiot!" - my lawyer 2005 when heard of Jan's saying above

lafleon7-Dec-06 10:25

lafleon

7-Dec-06 10:25

Hello,

.NET interop service marshals all interfaces automatically for all new threads. Marshaling fails for incorrect data types in the interface. You can only use automation supported data types like BSTR, VARIANT_BOOL, SAFEARRAY etc. For complete list of data types see VS documentation on MSDN or ATL wizard in Visual Studio.

Regards

anderslundsgard7-Dec-06 2:40

anderslundsgard

7-Dec-06 2:40

Ok, I've seen that function before.

I programming in .NET (C#). What type of method can I use in .NET.

I've run into the Control.Invoke method but I have about 30 calls to my TreeView and it is very time-consuming to rewrite all these calls...

_____________________________

...and justice for all

APe

Roger Stoltz7-Dec-06 3:22

Roger Stoltz

7-Dec-06 3:22

d00_ape wrote:
I programming in .NET (C#). What type of method can I use in .NET.

Sorry, I neither do .NET nor C#.

d00_ape wrote:
I've run into the Control.Invoke method but I have about 30 calls to my TreeView and it is very time-consuming to rewrite all these calls...

Why would all those calls have to be rewritten?
This cannot have anything to do with eventual calls to the marshalling API.

--
Roger

anderslundsgard7-Dec-06 4:01

anderslundsgard

7-Dec-06 4:01

Roger Stoltz wrote:
This cannot have anything to do with eventual calls to the marshalling API

Sorry I don't know what you mean with that...

I've posted this topic in the C# forum.

_____________________________

...and justice for all

APe

Roger Stoltz7-Dec-06 4:50

Roger Stoltz

7-Dec-06 4:50

d00_ape wrote:
Roger Stoltz wrote:

This cannot have anything to do with eventual calls to the marshalling API

Sorry I don't know what you mean with that...

I meant that it doesn't matter whether the code is implemented in C# or C++.
If adding marshalling of an interface would require "every call to be rewritten", then I suspect either a design flaw or the API to be used in a way it wasn't intended.
I hope you don't mean that "rewriting a call" means changing the name of your interface variable and you consider a "search'n'replace" for changing the name of a variable time consuming. Roll eyes | :rolleyes:

Why do you think that every call needs to be rewritten?
Why do you create a second thread and what does it do?
My questions are language independant.

--
Roger

CocreateInstance fails

Shailesh Ha5-Dec-06 19:04

5-Dec-06 19:04

Hi,
Problem is that A call to CocreateInstance fails ....hence the requested interface pointer shown is NULL ...hrStatus = CoCreateInstance(

clsControl,

NULL,

CLSCTX_INPROC_SERVER,

IID_Control,

reinterpret_cast<lpvoid *="">(&m_pControl)

);

ABOVE the IID_Control is showing as NULL...
Any solutions , Kindly let me know

regards
Shailesh

prasad_som5-Dec-06 20:22

5-Dec-06 20:22

Shailesh Halankar wrote:
ABOVE the IID_Control is showing as NULL...

I think, you wants to say, m_pControl is NULL.

Can you show complete code ?
Have you used ::CoInitialize(NULL) before calling CoCreateInstance?

Prasad
Notifier using ATL | Operator new[],delete[][^]

Shailesh Ha5-Dec-06 21:55

5-Dec-06 21:55

Hello
ur right m_pControl is NULL ..hence error..
I can giv u some more of the code ...

HRESULT hrStatus = CLSIDFromProgID( TEXT("What ever prog ID"), &clsControl );

if( SUCCEEDED(hrStatus) )
{
//
// Create the object and initialize it.
//

hrStatus = CoCreateInstance(
clsControl,
NULL,
CLSCTX_INPROC_SERVER,
IID_IControl,
reinterpret_cast<lpvoid *="">(&m_pControl)
);

if( SUCCEEDED(hrStatus) )
{

// some code
}
}

prasad_som5-Dec-06 22:15

5-Dec-06 22:15

Shailesh Halankar wrote:
HRESULT hrStatus = CLSIDFromProgID( TEXT("What ever prog ID"), &clsControl );

if( SUCCEEDED(hrStatus) )

I think you are getting class id here.
Call, ::Coinitialize(NULL) before it.

Shailesh Halankar wrote:
hrStatus = CoCreateInstance(
clsControl,
NULL,
CLSCTX_INPROC_SERVER,
IID_IControl,
reinterpret_cast(&m_pControl)
);

What is value of hrstatus here?

Prasad
Notifier using ATL | Operator new[],delete[][^]

Shailesh Ha5-Dec-06 22:31

5-Dec-06 22:31

Hi Prasad
I cant use coinitialize function ..gives linker error..i think its coz its not a VC++ apllication that i m working on . Its a Embeded VC++ application that i m working on.

regards
Shailesh

prasad_som5-Dec-06 22:40

5-Dec-06 22:40

Shailesh Halankar wrote:
..gives linker error

You need to link Ole232.lib.

Shailesh Halankar wrote:
i think its coz its not a VC++ apllication that i m working on . Its a Embeded VC++ application that i m working on.

Doesn't matter.

BTW, you haven't answered my second question , about value of hresult.

Prasad
Notifier using ATL | Operator new[],delete[][^]

Stephen Hewitt5-Dec-06 20:27

Stephen Hewitt

5-Dec-06 20:27

Your code is hard to read. What is the HRESULT returned? Try entering "hr, hr" in your debugger watch window, assuming the variable containing the HRESULT returned from CoCreateInstance is named "hr". If it was named "MyHr" you'd use the following string: "MyHr, hr".

Steve

Shailesh Ha5-Dec-06 23:17

5-Dec-06 23:17

my code ..is somewat like this

HRESULT hrStatus = CLSIDFromProgID( TEXT("some"), &clsControl );

if( SUCCEEDED(hrStatus) )
{
//
// Create the object and initialize it.
//

hrStatus = CoCreateInstance(
clsControl,
NULL,
CLSCTX_INPROC_SERVER,
IID_IControl,
reinterpret_cast<lpvoid *="">(&m_pControl)
);

if( SUCCEEDED(hrStatus) )
{
//some code
}
}

regards
Shailesh