Debugging code

[skooter]

Hi! New to the forum, wondering if anyone could help me?

I'm getting this error when trying to compiler the following code on Cygwin...

header.h:14: warning declaration does not declare anything
header.h:14 warning: no semi-colon at end of struct or union
header.h:14 error: parse error before '[' token
header.h:16 parse error before '}' token for the following lines of code:

12 typedef struct
13 {
14 char[BUFSIZE] filename;
15 long size;
16 } FINDFILE;

I'm running -std=c99 -Wall and -Werror

Any thoughts???
xxsKoot

[Banfa]

char[BUFSIZE] filename;

should be

char filename[BUFSIZE];

[skooter]

DOH! *slap*

ok next question: what is the makefile file extension?? (mak??) notepad seems to be saving it as Makefile.mak.txt... i've tried Makefile.txt and Makefile.exe... are these Make commands ok:

C99 = gcc -std=c99
FLAGS = -Wall -Werror -pedantic

mp3search.exe : mp3search.o globals.o fucts.o
$(C99) $(FLAGS) -o mp3search.exe mp3search.o globals.o fucts.o

mp3search.o : mp3search.c header.h
$(C99) $(FLAGS) -c -o mp3search.o mp3search.c

globals.o : globals.c header.h
$(C99) $(FLAGS) -c -o globals.o globals.c

fucts.o : fucts.c header.h
$(C99) $(FLAGS) -c -o fucts.o fucts.c

[Banfa]

ok next question: what is the makefile file extension?? (mak??) notepad seems to be saving it as Makefile.mak.txt... i've tried Makefile.txt and Makefile.exe... are these Make commands ok:

Actually anything you want to use but *.mak and *.mk are common, makefile. (i.e. no extension) is also common.

In notepad you need to switch the file type to all files before saving otherwise notepad adds a .txt

C99 = gcc -std=c99
FLAGS = -Wall -Werror -pedantic

mp3search.exe : mp3search.o globals.o fucts.o
$(C99) $(FLAGS) -o mp3search.exe mp3search.o globals.o fucts.o

mp3search.o : mp3search.c header.h
$(C99) $(FLAGS) -c -o mp3search.o mp3search.c

globals.o : globals.c header.h
$(C99) $(FLAGS) -c -o globals.o globals.c

fucts.o : fucts.c header.h
$(C99) $(FLAGS) -c -o fucts.o fucts.c

these look ok to me.

[skooter]

thanks, how about this one:


DIR *dirp1;
struct dirent *dp1;

dirp1 = opendir(searchStart);

then there is a perror check which is returning NULL, even though the same directory pathway(searchStart) is opening fine in another file...

[skooter]

Scratch that (too late to edit)

what does opendir: too many open files mean?? that I've run out of memory to store file pointers??

[skooter]

Okay, opendir is throwing the error because the for(dp1 = readdir(dirp1) ; ...) statement only seems to be reading the first file ("."). I've got an if-then-continue statement in, yet it doesnt seem to read past this point...

Any thoughts?

[Banfa]

You realise that the readdir statement is in the initialisation expression and is therefore on executed once before the loop starts running?

To read all the dirs I guess (because I can't see your code) that you will be needing to call readdir for every iteration of the loop.