Is it legal do declare this where you do? I'm mostly an embedded guy and this kind of dynamic declaration is verboten. The compiler seems happy with it, but I dunno...
Also it's dilemma not dilema (misspellings like this can get you into trouble...)Code:int indexP1[size/2], indexP2[size/2];
It is legal with respect to the 1999 edition of the C standard.
Look up a C++ Reference and learn How To Ask Questions The Smart WayOriginally Posted by Bjarne Stroustrup (2000-10-14)
OK, cool.
The third term should be something that changes 'd'. This doesn't causing the for loop to blow up.Code:for(d=0; d<size-cols; d+cols)
You still have no exit for no file found. May I suggest:
Code:if(file == NULL) { printf("Sorry! The file could not be opened! Press a key to continue\n"); getchar(); return 1; }
Last edited by mike65535; 04-20-2011 at 06:21 AM.
I get the following warnings when I compile this code:
You do have warnings enabled, right?main.c|13|warning: return type defaults to ‘int’|
main.c||In function ‘main’:|
main.c|54|warning: statement with no effect|
||=== Build finished: 0 errors, 2 warnings ===|
And line 54 is:Look closely at the last section of this statement (d+cols), this is a problem.Code:for(d=0; d<size-cols; d+cols){
Jim
Ha! I actually missed that until after I'd done several rewrites of the code and just left it that way because... laziness.
I caught the "d+cols" mistake as soon as I took a look at that line alone (somehow that always helps). Also, how do you enable warnings? or check?
This depends on how you are compiling. What compiler or IDE are you using, and what operating system?Also, how do you enable warnings?
Jim
I run ubuntu and I compile in the terminal using gcc.
Then pass -Wall as an option when invoking the compiler. -pedantic and -std=c99 or -ansi may also come in handy.
Look up a C++ Reference and learn How To Ask Questions The Smart Way
