SDL program problem...

slashdot · 07-22-2005, 12:47 PM

This program doesn't compile right... I believe its because it doesn't have a proper return 0; at the end. But when I put it at the end, it gives me the same error. (error at bottom)

Code:

#include <iostream>
#include <cstdlib>
#include <cstdio>
#include <SDL/SDL.h> 
using namespace std;

int main(int nNumberofArgs, char* pszArgs[])
{
	if(SDL_Init(SDL_INIT_VIDEO) < 0 ) {
		fprintf(stderr, "Could not initialize SDL: %s\n",
				SDL_GetError());
			return -1;
}

atexit(SDL_Quit);

SDL_Surface* screen;
screen = SDL_SetVideoMode(640, 480, 16, SDL_SWSURFACE|SDL_ANYFORMAT);
if( !screen ) {
	fprintf(stderr, "Couldn't create a surface: %s\n",
			SDL_GetError());
	return -1;
}

Quote:

ERROR:
anyone@george's:~$ g++ -s SDLBLECH.cpp -o SDLBLECH `sdl-config --libs --cflags`
SDLBLECH.cpp: In function `int main(int, char**)':
SDLBLECH.cpp:26: error: parse error at end of input
anyone@george's:~$

How would I fix this problem? I am just playing with SDL to understand it better, just if you want to know what I was doing.

redhead · 07-22-2005, 12:57 PM

Here is your problem:

Code:

int main(int nNumberofArgs, char* pszArgs[])
{
	if(SDL_Init(SDL_INIT_VIDEO) < 0 ) {
		fprintf(stderr, "Could not initialize SDL: %s\n",
				SDL_GetError());
			return -1;
}

You missed a ending } for your if() statement.
This should be better:

Code:

int main(int nNumberofArgs, char* pszArgs[])
{
	if(SDL_Init(SDL_INIT_VIDEO) < 0 ) {
		fprintf(stderr, "Could not initialize SDL: %s\n",
				SDL_GetError());
			return -1;
             }
      return 0;
}

Altho the ending part of it, dosn't make any sence

Code:

atexit(SDL_Quit);

SDL_Surface* screen;
screen = SDL_SetVideoMode(640, 480, 16, SDL_SWSURFACE|SDL_ANYFORMAT);
if( !screen ) {
	fprintf(stderr, "Couldn't create a surface: %s\n",
			SDL_GetError());
	return -1;
}

is left floating in mid air. You are not allowed to have an if() statement standing alone, it must be encapsulated by some form of function, from where you can activate that code in the simple function call elsewhere in your code. If I were you, I would merge that with the main() function, so entire code would be something like this:

Code:

#include <iostream>
#include <cstdlib>
#include <cstdio>
#include <SDL/SDL.h> 
using namespace std;

int main(int nNumberofArgs, char* pszArgs[])
{
    if(SDL_Init(SDL_INIT_VIDEO) < 0 ) {
        fprintf(stderr, "Could not initialize SDL: %s\n",SDL_GetError());
        return -1;
    }
    atexit(SDL_Quit);
    SDL_Surface* screen;
    screen = SDL_SetVideoMode(640, 480, 16, SDL_SWSURFACE|SDL_ANYFORMAT);
    if( !screen ) {
        fprintf(stderr, "Couldn't create a surface: %s\n", SDL_GetError());
        return -1;
    }
    return 0;
}

slashdot · 07-22-2005, 01:51 PM

Using your example I get this error...

anyone@george's:~$ g++ -s SDLBLECH.cpp -o SDLBLECH `sdl-config --libs --cflags`
SDLBLECH.cpp:1: error: parse error before `=' token
SDLBLECH.cpp:1: error: parse error before `#' token
In file included from /usr/include/c++/3.3.4/cstdlib:50,
from SDLBLECH.cpp:2:
/usr/include/c++/3.3.4/cstddef:52: error: `ptrdiff_t' not declared
anyone@george's:~$ ls

slashdot · 07-22-2005, 05:30 PM

AH! I got it to work...

Code:

{
	SDL_Surface *screen;

	printf("Initializing SDL!\n");

	if((SDL_Init(SDL_INIT_VIDEO)==-1))
	{
		printf("Could not initialize SDL: %s\n: SDL_GetError()");
			exit(-1);
	}

	screen = SDL_SetVideoMode(640, 480, 8, SDL_SWSURFACE);	
	if ( screen == NULL )
	{
		fprintf(stderr, "Couldn't set 640x480 video mode: %s\n", SDL_GetError());
		exit(1);
	}

Thanx for the help though.. you put in the right direction.