My DLL not calling base class

Question

1.00/5 (1 vote)

See more:

I have an ext dll which has a base class, It has been simplified so could see what going on Header file defined as

#include "CmnStruct.h"
class B10 CBYetiBase : public CmnStruct
{
public:
	// main app/errcontrol
	YVoid CreateErrControlStruct(sAppParam *sParams, PROTO_ERR_CODES);
};

in Source Code

YVoid CBYetiBase::CreateErrControlStruct(sAppParam *sParams, PROTO_ERR_CODES)
{
	FNC_INITIAL(CreateErrControlStruct); L_RET_PTRNULL(sErrControl);
	//                          ***
	int n = 0;          
	n = CmnStruct::Return();
	//
	RETASYVOID;             //  ***
}

this is all thats in CmnStruct

class CmnStruct
{
public:
	int Return();

};

int CmnStruct::Return()
{
	return LB_ERR;
}

Most of the words are defined macros and work else where when run it jumps line between marked ***

What I have tried:

Have tried making the class CmnStruct to use -> and . as function calls
the only way it works is by removing the class and putting the code directly into CBYetiBase::CreateErrControlStruct but I need to reference other functions from the Cmn class (the function used here, has been simplified)

Posted 3-Aug-23 22:22pm

Idd Jutt 2021

Updated 4-Aug-23 3:56am

Add a Solution

Comments

CPallini 4-Aug-23 5:30am

How do you check that?

Idd Jutt 2021 4-Aug-23 8:20am

Sorry how do I check what ?

CPallini 4-Aug-23 8:22am

That code was actually skipped.

Idd Jutt 2021 4-Aug-23 8:23am

single stepping after a breakpoint

CPallini 4-Aug-23 9:23am

Such a code has no effect. It could have been optimized out.

Idd Jutt 2021 4-Aug-23 9:55am

My fault, it was optimized out, I did not notice it was in Release once I switched back to Debug, then the thing worked, Wondered why it was working couple months ago

Idd Jutt 2021 4-Aug-23 9:58am

thanx for assist made me look closer at the settings

CPallini 4-Aug-23 10:17am

You are welcome.

CPallini 4-Aug-23 8:22am

Also note that:

	int n = 0;          
	n = CmnStruct::Return();

has no effect and could have been optimized out.

Idd Jutt 2021 4-Aug-23 8:26am

Would it optimise out if the call did a return of LB_ERR, perhaps I will try putting in my function initializers just to add more code

Idd Jutt 2021 4-Aug-23 8:34am

Put more code within function and no change jumps over the call

2 solutions

Solution 1

All those macros make your code very difficult to understand. However, as far as I can see, that code will not even compile. You are trying to call a non-static class method as if it was declared static. You need to create an instance of the CmnStruct class and call Return through it, thus:

C++

CmnStruct mystruct; int n = 0; n = mystruct.Return();

[edit]
As per CPallini's reply below.
[/edit]

Posted 4-Aug-23 1:46am

Richard MacCutchan

Updated 4-Aug-23 3:20am

v3

Comments

CPallini 4-Aug-23 7:55am

Nope, it is not an attempt to call a static method.
It is a call to its own instance method (after all 'CBYetiBase' is a 'CmnStruct').
Try

#include <iostream>
  
using namespace std;

class Base
{
protected:
int the_answer(){ return 42;}

public:
  Base(){}
};

class Derived: public Base
{
public:
  Derived(){ cout << "The answer is " << Base::the_answer() << "\n";};
};


int main()
{
  Derived d;
}

Idd Jutt 2021 4-Aug-23 8:15am

n = CmnStruct::Return();
I know I do not need the class specifier, I add them in for clarity

Richard MacCutchan 4-Aug-23 8:40am

Thank you, I misread that.

Idd Jutt 2021 4-Aug-23 8:07am

Yep already tried that, it still jumps over, and the variable n will as unrecognised, and yes do use a lot of macros, but they are mainly setting up initial values, and a return value set as a type

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Idd Jutt 2021 · Accepted Answer · 2023-08-04T03:56:00

Solution 2

the solution was to put back in Debug mode, some how was in Release

Posted 4-Aug-23 3:56am

Idd Jutt 2021