Discussion :

[Grulim]

Bonjour,

J'ai une erreur sous GCC 4.2.0 à la compilation :
Citation:
error: variably modified 'sDescription' at file scope
et le bout de code qui provoque ça est une simple déclaration:
Code :

Code :

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

1
2
3
4
5
static const int ETAPIMP_MAX_BUFFER = 1020;
typedef struct _import {
  char cType;
  char sDescription[ETAPIMP_MAX_BUFFER + 1];
} Import_t;

1) Que veut dire exactement ce message d'erreur ?
2) Pourquoi est-ce une erreur ?

Merci.

[ram-0000]

Je pense que cela ne plait pas à ton compilateur que la taille du champs sDescription soit défini par une variable et pas un constante.

ETAPIMP_MAX_BUFFER est une variable (certes, déclarée constante) mais une variable quand même

[homeostasie]

Tu es entrain de réaliser un VLA (Variable Length Array), je ne pense pas que c'est cela que tu désires faire.

L'erreur provient du fait que tu ne peux définir un VLA au niveau du "file scope". Tu dois le définir dans un "block scope", par exemple comme suit:

Code :

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

1
2
3
4
5
6
7
void func(int n)
{
     typedef int A[n]; 
     A a;
 
     ...
}

ou

Code :

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

1
2
3
4
5
6
void func(int n)
{
     int tab[n]; 
 
     ...
}

A noter que cela a été introduit par la norme C99.

Sinon, dans ton cas, tu pourrais remplacer ceci:

Code :

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

static const int ETAPIMP_MAX_BUFFER = 1020;

par

Code :

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

#define ETAPIMP_MAX_BUFFER   1020;

[Grulim]

Merci de vos éclaircissements.

J'ai fait 2 essais :

Code :

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

1
2
3
4
5
6
void foo()
{
  static const int CSTE = 20;
  char toto1[CSTE];
  char toto2[CSTE + 1];
}

qui compile.
et celui-la :

Code :

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

1
2
3
4
5
6
static const int CSTE = 20;
char toto1[CSTE];
char toto2[CSTE + 1];
void foo()
{
}

qui ne compile pas.
Donc c'est un pb de VLA (je pensais que ça pouvait être le "+ 1" qui embête le compilo).
Malgré vos explications (et même si je parais présomptueux), cette erreur me parait... illogique.
Ou alors, dans l'exemple 2, CSTE n'est pas assuré d'être initialisé avant les tableaux, ce qui expliquerait pourquoi toto1 et toto2 soient traités comme des VLA.

[nicolas.sitbon]

Array objects declared with the static or extern storage-class specifier
cannot have a variable length array (VLA) type.

[ram-0000]

Array objects declared with the static or extern storage-class specifier
cannot have a variable length array (VLA) type.

Je crois que c'est clair ...

[Grulim]

Si j'ai bien compris, l'exemple suivant est valide :

Code :

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

1
2
3
4
5
6
static const int CSTE = 20;
void foo()
{
  char toto1[CSTE]; /* <- array object ni static ni extern*/
  char toto2[CSTE + 1]; /* <- array object ni static ni extern*/
}

(gcc le compile en tout cas... mais ce n'est pas suffisant)

[nicolas.sitbon]

Si j'ai bien compris, l'exemple suivant est valide :

Code :

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

1
2
3
4
5
6
static const int CSTE = 20;
void foo()
{
  char toto1[CSTE]; /* <- array object ni static ni extern*/
  char toto2[CSTE + 1]; /* <- array object ni static ni extern*/
}

Oui, d'ailleurs, CSTE n'a même pas besoin d'être constant pour que la déclaration reste valide.

[homeostasie]

Donc c'est un pb de VLA (je pensais que ça pouvait être le "+ 1" qui embête le compilo).
Malgré vos explications (et même si je parais présomptueux), cette erreur me parait... illogique.
Ou alors, dans l'exemple 2, CSTE n'est pas assuré d'être initialisé avant les tableaux, ce qui expliquerait pourquoi toto1 et toto2 soient traités comme des VLA.

Je m'avance peut être (rectifiez moi si je me trompe), mais je pense que cela vient du fait qu'un VLA est évidemment déterminé au moment de l'exécution, comme pour une allocation dynamique, et non au moment de la compilation comme cela aurait été pour un tableau statique.