Solved

data conversion error when assigning function to function pointer

Posted on 2007-03-21
7
751 Views
Last Modified: 2013-12-08
Hi Experts,
    I am trying to assign a function to a function pointer member in CFactoryTemplate, which is a class from directshow SDK

this is the class that i wrote:
class CMYFilter: public CTransInPlaceFilter
{
    public:
        CUnknown * WINAPI CreateInstance(LPUNKNOWN pUnk, HRESULT *pHr);
};

What i am trying to do is assign CreateInstance to a member of type
LPFNNewCOMObject
which has its original declaration as
typedef CUnknown *(CALLBACK *LPFNNewCOMObject)(LPUNKNOWN pUnkOuter, HRESULT *phr);

the assignment is shown here:
CFactoryTemplate g_Templates[1] =
{
  {
    g_wszName,
    &CLSID_CMYFilter,
    CMYFilter::CreateInstance,  // line with error
    NULL,
    NULL
  }
};

I kept receiving this error message at the line CMYFilter::CreateInstance.

error C2440: 'initializing' : cannot convert from 'CUnknown *(__cdecl
CMYFilter::* )(LPUNKNOWN,HRESULT *)' to 'LPFNNewCOMObject'
1>        None of the functions with this name in scope match the target type

This is very strange since all the sample codes, including those came with
SDK, were written like this.

Is there any reason why this is happening?
thanks!
0
Comment
Question by:fungi8210
  • 5
  • 2
7 Comments
 
LVL 86

Accepted Solution

by:
jkr earned 500 total points
ID: 18766235
You need to make that function a 'static' member of your class - non static ones are passed an invisible 'this' pointer which makes the signature different, thus the error.

class CMYFilter: public CTransInPlaceFilter
{
    public:
        static CUnknown * WINAPI CreateInstance(LPUNKNOWN pUnk, HRESULT *pHr);
};

will get that to work.
0
 
LVL 86

Expert Comment

by:jkr
ID: 18766237
BTW, a factory function also does not need to be a non-static member.
0
 

Author Comment

by:fungi8210
ID: 18766258
i just added static to it, now i am getting these errors
error LNK2001: unresolved external symbol "public: static class CUnknown * __cdecl CFPSFilter::CreateInstance(struct IUnknown *,long *)" (?CreateInstance@CFPSFilter@@SAPAVCUnknown@@PAUIUnknown@@PAJ@Z)

error LNK2001: unresolved external symbol CLSID_FPSFilter
1>Windows Mobile 6 Professional SDK (ARMV4I)\Debug/MyFilter.dll : fatal error LNK1120: 2 unresolved externals

just incase you are wondering, i'm trying to write a directshow filter, and CLSID_FPSFilter is my classid defined at the top

// {033104B2-2AA9-4476-992E-AB504F9F1F1E}
DEFINE_GUID(CLSID_FPSFilter,
0x33104b2, 0x2aa9, 0x4476, 0x99, 0x2e, 0xab, 0x50, 0x4f, 0x9f, 0x1f, 0x1e);

btw, you answered my question correctly, you'll get the points, just wondering if you know how to solve my second problem, thanks!
0
Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

 
LVL 86

Expert Comment

by:jkr
ID: 18766283
You'll need to provide an implementation also, e.g.

static CUnknown * WINAPI CMYFilter::CreateInstance(LPUNKNOWN pUnk, HRESULT *pHr) {

  CMYFilter* p = NULL;

  // other code to create a 'CMYFilter' here...

  return p;
}

0
 
LVL 86

Expert Comment

by:jkr
ID: 18766293
Sorry, that should have been

CUnknown * WINAPI CMYFilter::CreateInstance(LPUNKNOWN pUnk, HRESULT *pHr) {

  CMYFilter* p = NULL;

  // other code to create a 'CMYFilter' here...

  return (CUnknown*) p;
}
0
 

Author Comment

by:fungi8210
ID: 18766319
thanks, one last error

CMYFilter.obj : error LNK2001: unresolved external symbol CLSID_CMYFilter
1>Windows Mobile 6 Professional SDK (ARMV4I)\Debug/MyFilter.dll : fatal error LNK1120: 1 unresolved externals

its weird because i defined CLSID_CMYFilter at the top


// {033104B2-2AA9-4476-992E-AB504F9F1F1E}
DEFINE_GUID(CLSID_CMYFilter,
0x33104b2, 0x2aa9, 0x4476, 0x99, 0x2e, 0xab, 0x50, 0x4f, 0x9f, 0x1f, 0x1e);

0
 
LVL 86

Expert Comment

by:jkr
ID: 18766368
>>its weird because i defined CLSID_CMYFilter at the top

Yes, but for that to be expanded to also an implementation (not just a declaration) you'll need to

#define INITGUID

before

#include <objbase.h>
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
Problem to show menu 5 87
C++ question 3 62
Header of docx file 17 98
How to install SVN Command Line Client? 5 43
Container Orchestration platforms empower organizations to scale their apps at an exceptional rate. This is the reason numerous innovation-driven companies are moving apps to an appropriated datacenter wide platform that empowers them to scale at a …
Performance in games development is paramount: every microsecond counts to be able to do everything in less than 33ms (aiming at 16ms). C# foreach statement is one of the worst performance killers, and here I explain why.
The goal of this video is to provide viewers with basic examples to understand opening and reading files in the C programming language.
The viewer will learn how to user default arguments when defining functions. This method of defining functions will be contrasted with the non-default-argument of defining functions.

867 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

26 Experts available now in Live!

Get 1:1 Help Now