Project

General

Profile

Actions

Task #446

closed

BP1 BP2 pour release 3

Added by paul leroy over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
22/06/2015
Due date:
% Done:

0%

Estimated time:
revision:

Description

Quelques remarques suite à mes premiers essais d'intégration des nouvelles fonctions de calcul dans la R3:

Les déclarations suivantes doivent être dans le .h, pas dans le .c
void init_k_coefficients_f0( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );
void init_k_coefficients_f1( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );
void init_k_coefficients_f2( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );

Dans ton switch/case, il faut utiliser des constantes non numériques définies en utilisant des #define, c'est une règle qui est vérifiée par l'outil de vérification statique utilisé par William Recart.

Actions #1

Updated by paul leroy over 9 years ago

Eviter au maximum de laisser des printf dans les codes définitif. Possibilité d'utiliser des macros:
#ifdef PRINT_TCH
#define PRINTF(x) printf(x);
#define PRINTF1 printf(x,y);
#define PRINTF2 printf(x,y,z);
#else
#define PRINTF(x) ;
#define PRINTF1 ;
#define PRINTF2 ;
#endif

Si la constante PRINT_TCH n'est pas définie, aucun message n'apparaît.

Autre point important: finir les messages par "\n"

Actions #2

Updated by thomas chust over 9 years ago

  • Assignee changed from thomas chust to paul leroy
Actions #3

Updated by paul leroy over 9 years ago

  • Status changed from New to Feedback
  • Assignee changed from paul leroy to thomas chust

Il faudrait inclure basic_parameters.h dans basic_parameters.c sinon ça fait des warning à la compilation.

#include <basic_parameters.h>

Actions #4

Updated by thomas chust over 9 years ago

  • Assignee changed from thomas chust to paul leroy

J'ai fait un nouveau push => version 2.2

En fait, me semble t-il, pas besoin. Dans basic_parameters.c il fallait simplement définir la fonction init_k_coefficients après les fonctions init_k_coefficients_f0, _f1 et _f2 ... Du coup cela permet aussi de supprimer les déclarations
void init_k_coefficients_f0( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );
void init_k_coefficients_f1( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );
void init_k_coefficients_f2( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );
dans le .h

D'accord ?

Actions #5

Updated by paul leroy over 9 years ago

  • Assignee changed from paul leroy to thomas chust

Ca marche si la fonction n'est pas destinée à être utilisée ailleurs que dans ton module basic_parameters.c.

Je ne sais pas si le vérificateur statique va tiquer en voyant une fonction non déclarée dans un .h. Dans le doute, je préfèrerais que les déclaration soit dans basic_parameters.h et la description dans basic_parameters.c.

Actions #6

Updated by thomas chust over 9 years ago

Entendu: push fait ...

Actions #7

Updated by thomas chust over 9 years ago

  • Assignee changed from thomas chust to paul leroy
Actions #8

Updated by paul leroy over 9 years ago

  • Status changed from Feedback to Closed
Actions

Also available in: Atom PDF