Bug #3124
closed
Analyse Logiscope LFR_3.2.0.21 : LFR_basic-parameters : Don_Initialisation_P1 Severity is High
0%
Description
Rappel de la règle :
Don_Initialisation_P1:
Definition:
-----------
Global variables must be initialized when they are defined.
Justification:
--------------
Not all compilers give the same default values. Unexpected behaviour can be
avoided with better control over variable values. Initializing global variables
when they are declared ensures that they are initialized before being used.
La règle n'est pas respectée dans 6 cas d'après Logiscope:
basic_parameters_utilities.h : lines : 19, 20, 21, 22, 24, 25
Updated by William Recart about 6 years ago
Exemple de correction :
float compressed_spectral_matrix_f0[NB_BINS_COMPRESSED_MATRIX_f0 * NB_VALUES_PER_SPECTRAL_MATRIX] = {0.0} ;
Updated by bruno katra about 6 years ago
- Status changed from New to In Progress
Corrigé, en attente de validation par Thomas.
Updated by Alexis Jeandet about 6 years ago
Ce source n'est pas utilisé dans le FSW, c'est juste utilisé pour les tests de Thomas un local sur son PC.
Les seuls sources à analyser et corriger sont:
- LFR_basic-parameters/basic_parameters.c
- LFR_basic-parameters/basic_parameters.h
- LFR_basic-parameters/basic_parameters_params.h
De plus le problème n'est pas tant l'init que la déclaration de variables dans un header...
Updated by bruno katra about 6 years ago
- Status changed from In Progress to Closed
Alexis Jeandet wrote:
Ce source n'est pas utilisé dans le FSW, c'est juste utilisé pour les tests de Thomas un local sur son PC.
Les seuls sources à analyser et corriger sont:
- LFR_basic-parameters/basic_parameters.c
- LFR_basic-parameters/basic_parameters.h
- LFR_basic-parameters/basic_parameters_params.hDe plus le problème n'est pas tant l'init que la déclaration de variables dans un header...
En effet ! J'avais pas fait attention que c'était un .h et en plus que ce fichier sert uniquement à Thomas en local.
On peut donc cloturer ce point puisque ce code ne fait pas partie du FSW...