Discussion :

[zenux]

Bonjour,

Quand je compile mon vertex shader, j'obtiens cette erreur:

0(1) : error C0000: syntax error, unexpected $undefined at token "@"
0(1) : error C0501: type name expected at token "@"
0(2) : warning C7530: OpenGL requires main to return void
0(2) : error C1110: function "main" has no return statement

Mon shader:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
void main()
{
	//glOrtho(-1.0, 1.0, -1.0, 1.0, 0.0, 1.0):		
	mat4 mModelView = mat4(
		1.0, 0.0, 0.0, 0.0, 
		0.0, 1.0, 0.0, 0.0,
		0.0, 0.0, -2.0, 0.0,
		0.0, 0.0, -1.0, 1.0);
 
	gl_TexCoord[0] = gl_MultiTexCoord0;
 
	gl_Position = mModelView * gl_Vertex;
}

Je ne comprends vraiment pas ce qui ne va pas. Quelques pistes:
1) Je lis mon vertex shader à partir d'un fichier texte et j'ai vérifié byte par byte ce que je lis: ça me semble correcte (pas de caractères bizarre, etc.) !
2) Plus étrange: j'ai en fait 2 shaders (2 vertex shader et 2 fragment shader). La compilation du premier shader ne retourne aucune erreur. Par contre la compilation du second shader me retourne une erreur (celle que j'ai mis au début de mon post).
Si j'inverse l'ordre de compilation: je compile d'abord mon second shader et ensuite le premier shader: c'est le premier shader qui me retourne une erreur (similaire à celle de ce post) !!!

Merci d'avance.

[AuraHxC]

Comment tu compile ton shader ?
Tu peux mettre le code qui fait cela ?

[zenux]

Voici le code qui compile le shader:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
unsigned int shaderID = glCreateProgramObjectARB();
 
//vertex shader
const char *vertexShaderSource = readEntireFile(vertexShaderFilename).c_str();
unsigned int vertexShader = glCreateShaderObjectARB(GL_VERTEX_SHADER_ARB);
glShaderSourceARB(vertexShader, 1, &vertexShaderSource, NULL);
glCompileShaderARB(vertexShader);
printLog(vertexShader, vertexShaderFilename);
glAttachObjectARB(shaderID, vertexShader);
 
//fragment shader
const char *fragmentShaderSource = readEntireFile(fragmentShaderFilename).c_str();
unsigned int fragmentShader = glCreateShaderObjectARB(GL_FRAGMENT_SHADER_ARB);
glShaderSourceARB(fragmentShader, 1, &fragmentShaderSource, NULL);
glCompileShaderARB(fragmentShader);
printLog(fragmentShader, fragmentShaderFilename);
glAttachObjectARB(shaderID, fragmentShader);
 
glLinkProgramARB(shaderID);
printLog(shaderID, vertexShaderFilename + "(link)");

Lecture du fichier:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
string ShaderManager::readEntireFile(const string &filename)
{
	string fileContent;
	fileContent.reserve(10000);
 
	ifstream file;
	file.open(filename.c_str(), ios::in|ios::binary);
	if(file.fail())
	{
		throw FileException(filename, "Cannot open this file.");
	}
 
	char c;
	while (file.get(c))
	{
		fileContent.append(1, c);
	}
 
	file.close();
 
	return fileContent;
}

[TanEk]

Ton objet string que tu retournes avec ta fonction de lecture de fichier n'est pas garde quelque part... En gros au moment ou tu appelles ta fonction ton objet string est detruit. Tu essaies ensuite d'acceder a une zone memoire desallouee qui peut etre ensuite etre reutilisee par d'autres objets qui vaut mettre des @m etc. Pour fixer cela fait:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
const string fragmentShaderSource = readEntireFile(fragmentShaderFilename);
unsigned int fragmentShader = glCreateShaderObjectARB(GL_FRAGMENT_SHADER_ARB);
glShaderSourceARB(fragmentShader, 1, &fragmentShaderSource._c_str(), NULL);

Edit: petite coquille dans le code ci-dessus explique dans les posts qui suivent.

[zenux]

Bien vu Problème résolu: merci.

Par contre, j'ai une autre petite question:
Ce code source compile:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
const string fragmentShaderSource = readEntireFile(fragmentShaderFilename);
const char *fragmentShaderSourceChar = fragmentShaderSource.c_str();
glShaderSourceARB(fragmentShader, 1, &fragmentShaderSourceChar, NULL);

Pas celui-ci (avec ou sans les parenthèses):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
const string fragmentShaderSource = readEntireFile(fragmentShaderFilename);
glShaderSourceARB(fragmentShader, 1, &(fragmentShaderSource.c_str()), NULL);

Erreur: "lvalue required as unary ‘&’ operand."
Il ne serait pas un peu con mon compilateur (ou alors c'est moi le con ) ? (g++)

[TanEk]

Non c'est le meme principe, la fonction c_str() retourne un pointeur de char et toi tu donnes l'adresse de ce pointeur en memoire qui est lui-meme... temporaire... Il est detruit des que tu rentres dans la fonction invalidant l'adresse le referencant que tu viens juste de passer a la fonction .

Il faut donc que tu crees ton pointeur avant d'appeler la fonction pour etre sur qu'il soit toujours valide quand tu rentres a l'interieur de la fonction.