compilation problems!!

[seema]

I have problem compiling this program using GCC(g++)

//bye.h --header file
#ifndef BYE_H
#define BYE_H
void display();
#endif

//bye.C -------- Source
#include "bye.h"
#include <iostream.h>

void display()
{
cout <<"Display the stuff ";

}

//main.cpp -----source
#include <iostream.h>
#include <string.h>
#include "bye.h"

class myclass
{
//mystruct x;
friend void display();
public:
myclass(){
display();
}
};

int main()
{
myclass me;

}

If I compile using g++ main.cpp I get the following error,

/tmp/ccG3OvUb.o: In function `myclass::mycla ss[in-charge]()':
/tmp/ccG3OvUb.o(.gnu .linkonce.t._ZN 7myclassC1Ev+0x 7): undefined
reference to `display()'
collect2: ld returned 1 exit status

Can some body explain how to do it
thanks in advance,
Seema Rao

[Phlip]

"seema" <se********@yah oo.co.inwrote in message
news:11******** **************@ 35g2000cwc.goog legroups.com...

>I have problem compiling this program using GCC(g++)

//bye.C -------- Source

The .C is probably triggering g++ to compile this as a C module.

#include "bye.h"
#include <iostream.h>

An unrelated issue; you should switch to <include>.

void display()
{
cout <<"Display the stuff ";

I suspect g++ can then compile C++ things in a C module. But that is
platform-specific, so you should ask a g++ forum.

class myclass
{
//mystruct x;
friend void display();
public:
myclass(){
display();
}
};

Can you also just call display() directly from main(), without the friend?

/tmp/ccG3OvUb.o: In function `myclass::mycla ss[in-charge]()':
/tmp/ccG3OvUb.o(.gnu .linkonce.t._ZN 7myclassC1Ev+0x 7): undefined
reference to `display()'

Google for "name mangling". I suspect the .C extension has mangled the name
display() for C, not C++. Linkers generally only link mangled names that
match.

--
Phlip
http://c2.com/cgi/wiki?ZeekLand <-- NOT a blog!!!

[Victor Bazarov]

seema wrote:

[..]
If I compile using g++ main.cpp I get the following error,

/tmp/ccG3OvUb.o: In function `myclass::mycla ss[in-charge]()':
/tmp/ccG3OvUb.o(.gnu .linkonce.t._ZN 7myclassC1Ev+0x 7): undefined
reference to `display()'
collect2: ld returned 1 exit status

Can some body explain how to do it

Somebody in 'gnu.g++.help' should be able to. You must be compiling
those translation units separately. You need to "link" them together
during linking. How to do that is compiler-specific and off-topic.

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[Thomas Tutone]

seema wrote:

I have problem compiling this program using GCC(g++)

//bye.h --header file
#ifndef BYE_H
#define BYE_H
void display();
#endif

//bye.C -------- Source

change the name of the file to bye.cpp - it will make your life much
easier if you are consistent.

#include "bye.h"
#include <iostream.h>

Change the above line to:
#include <iostream>

void display()
{

add the following line here:
using namespace std;

cout <<"Display the stuff ";

}

//main.cpp -----source
#include <iostream.h>

Change the above line to:
#include <iostream>

#include <string.h>

Delete the above line - you don't need that header in this program.

#include "bye.h"

class myclass
{
//mystruct x;
friend void display();

Delete the above line - you don't need it in this program.

public:
myclass(){
display();
}
};

int main()
{
myclass me;

}

If I compile using g++ main.cpp I get the following error,

/tmp/ccG3OvUb.o: In function `myclass::mycla ss[in-charge]()':
/tmp/ccG3OvUb.o(.gnu .linkonce.t._ZN 7myclassC1Ev+0x 7): undefined
reference to `display()'
collect2: ld returned 1 exit status

Can some body explain how to do it

<implementati on specific/>

If you are on Linux, compile it as follows:

g++ -Wall -Wextra -pedantic main.cpp bye.cpp -o seema

and execute it by typing:

seema

If you are on Windows, compile it as follows:

g++ -Wall -Wextra -pedantic main.cpp bye.cpp -o seema.exe

and execute it by typing:

seema

</implementation specific>

Hope that helps.

Best regards,

Tom

[Howard]

"Phlip" <ph******@yahoo .comwrote in message
news:iG******** ***********@new ssvr12.news.pro digy.com...

>
"seema" <se********@yah oo.co.inwrote in message
news:11******** **************@ 35g2000cwc.goog legroups.com...

>>I have problem compiling this program using GCC(g++)

>//bye.C -------- Source

The .C is probably triggering g++ to compile this as a C module.

>#include "bye.h"
#include <iostream.h>

An unrelated issue; you should switch to <include>.

huh?

[Victor Bazarov]

Howard wrote:

"Phlip" <ph******@yahoo .comwrote in message
news:iG******** ***********@new ssvr12.news.pro digy.com...

>>
"seema" <se********@yah oo.co.inwrote in message
news:11******* *************** @35g2000cwc.goo glegroups.com.. .

>>I have problem compiling this program using GCC(g++)

>>//bye.C -------- Source

The .C is probably triggering g++ to compile this as a C module.

>>#include "bye.h"
#include <iostream.h>

An unrelated issue; you should switch to <include>.

huh?

I think "Phlip" meant <iostream>

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[Rolf Magnus]

Thomas Tutone wrote:

If you are on Linux, compile it as follows:

g++ -Wall -Wextra -pedantic main.cpp bye.cpp -o seema

-Wextra is only available on quite recent versions. Older ones want -W
instead. I would also add -ansi.

and execute it by typing:

seema

On a sane installation, this won't work. Better try:

./seema

[Thomas Tutone]

Rolf Magnus wrote:

Thomas Tutone wrote:

If you are on Linux, compile it as follows:

g++ -Wall -Wextra -pedantic main.cpp bye.cpp -o seema

-Wextra is only available on quite recent versions. Older ones want -W
instead. I would also add -ansi.

and execute it by typing:

seema

On a sane installation, this won't work. Better try:

./seema

Quite right. Thanks for the correction.

Best regards,

Tom