Bug #3124
closed
Analyse Logiscope LFR_3.2.0.21 : LFR_basic-parameters : Don_Initialisation_P1 Severity is High
Added by William Recart over 5 years ago.
Updated over 5 years ago.
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
Exemple de correction :
float compressed_spectral_matrix_f0[NB_BINS_COMPRESSED_MATRIX_f0 * NB_VALUES_PER_SPECTRAL_MATRIX] = {0.0} ;
- Status changed from New to In Progress
Corrigé, en attente de validation par Thomas.
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...
- 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.h
De 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...
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by