Printable View
OK im getting one more error while compiling
Undefined first referenced
symbol in file
main /usr/sfw/lib/gcc/sparc-sun-solaris2.10/3.4.3/crt1.o
ld: fatal: Symbol referencing errors. No output written to a.out
collect2: ld returned 1 exit status
...
THere is no line number given.What does this mean?
It means you're referencing an identifier that has no definition that it can see. Why don't you post your code?
Its a huge code
Code:
#include<string.h>
#include <stdio.h>
#include <stdlib.h>
#include "stack.h"
/*
* *********************************StackInit
* A new stack variable is initialized. The initialized
* stack is made empty.
* INPUT: stackP
* OUTPUT:void
*/
void StackInit(stackST *stackP)
{
stackElement *newContents;
/* Allocate a new array to hold the contents. */
newContents = (stackElement *)malloc(sizeof(stackElement) * 10);
stackP->contents = newContents;
stackP->lineST = 0;
stackP->top = -1; /* I.e., empty */
}
/*
* ******************************StackPush
* Add an element to the top of the stack
* INPUT: stackP. element and the line number where the tag was found
* OUTPUT:void
*
*/
void Push(stackST *stackP, stackElement element, int line)
{
/* Put information in array; update top. */
stackP->contents[++stackP->top] = element;
stackP->lineST=line;
}
/*
* ******************************StackPop
* Remove an element from the top of the stack.It first checks if the previous opening tag is closed,
* else it displays an error and recursively checks if the next opened tag has been closed.
* INPUT: stackP. element and the line number where the tag was found
* OUTPUT:void
*
*/
stackElement Pop(stackST *stackP, char *checkelement, int line)
{
if (strcmp(checkelement,stackP->contents)==0)
return stackP->contents[stackP->top--];
else
{
printf("\n Error: unmatched tag </%s>in line %d corresponding to line %d",checkelement,line,stackP->lineST);
stackP->top--;
return Pop(stackP,checkelement,line);
}
}
Do you have a main function somewhere?
That's not your whole code. I don't see anything wrong with that file itself (that would cause a linker to throw errors: there are a few code issues (don't cast malloc()).
Suggestions:
1) You can attach code rather than pasting if you feel it is too big for a post.
2) You do have a main(), don't you?
3) Try a clean compile. (Perhaps with -Wall too)
This is my main method
Code:
#include <stdio.h>
#include <string.h>
#include "stack.h"
#include <stdlib.h>
int main(void)
{
stackST s1;
int line = 0;
char c;
char element[10]; /* Tag that is captured.*/
char newelement[10]; /* New element to be pushed.*/
char check[10]; /* Check tag that is sent to the pop function*/
int i=0;/*counters*/
int j,k=0;/**/
int *temp;/* temporary pointer to copy only the words in the tag leaving out unnecessary input like <>*/
char code[500];/* each line that is parsed from the input file*/
/*Initializes the stack*/
stackInit(&s1);
while (scanf("%s",code)!=EOF)
{
++line;
while (i<strlen(code))
{
c=code[i];
if (c == '<')
{
j=0;
do
{
c=code[i];
element[j++]=c;
code[i++];
}while(c!='>');
if (element[1] == '/') /* if close tag*/
{
*temp=element[2];
k=0;
while(k<=strlen(element)-3)
check[k++]=putchar(*temp++);
Pop(&s1, check,line);
printf("%s",element);
}
else if(element[1] == '!')/* if comments, that should be delted from the output*/
{}
else /* If opening tag*/
{
*temp=element[1];
k=0;
while(k<=strlen(element)-2)
newelement[k++]=putchar(*temp++);
Push(&s1,newelement,line);
printf("%s",element);
}
}
else
printf("%c",c);/* print everything that is not a tag*/
}
}
printf("\n \n \n File was successfully verified");
putchar('\n');
return 0;
}
stack.h
Code:
#include <stdlib.h>
typedef char stackElement; /* the type of the objects entered in the stack.*/
/* Stack structure*/
typedef struct {
stackElement *contents;
int top;
int lineST;
}stackST;
void StackInit (stackST *stackP);
void Push(stackST *stackP, stackElement element, int line);
stackElement Pop(stackST *stackP, char *checkelement, int line);
Why do you say not to use the malloc function?
He didn't say don't use malloc, he said don't cast malloc.
I'd say crank up the warnings level on your compiler and try to fix the compiler issues first (like what do you want this to do?).The changes mentioned in your other thread cascade through the code, so there are a number of places to fix.Code:do
{
c=code[i];
element[j++]=c;
code[i++];
}while(c!='>');
element needs to be a pointer, and that changes things as well.Code:void Push(stackST *stackP, stackElement element, int line)
{
/* Put information in array; update top. */
stackP->contents[++stackP->top] = element;
stackP->lineST=line;
}
Don't let this pointer dangle.Code:int *temp;/* temporary pointer to copy only the words in the tag leaving out unnecessary input like <>*/
I didnt understand what you meant bout thelast 2 changes. Please explain.
Thanks.
I mean I didnt understand the last error about the pointer dangling.
The pointer is not initialized. It points to nothing and something. It points to something that is nothing, yet does not fit the definition of nothing. It it a very profound concept, understood by very few. Mastered by even fewer. Ponder this. Study its meaning, Grasshopper.
Quzah.
That was beautiful, quzah. :o
Kind of reminds me of: http://www.educ.umu.se/~bjorn/misc/tao.big.html