https://hephaistos.lpp.polytechnique.fr/redmine/https://hephaistos.lpp.polytechnique.fr/redmine/redmine/favicon.ico?15080976012015-06-22T12:57:51ZRedmineLFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12022015-06-22T12:57:51Zpaul leroy
<ul></ul><p>Eviter au maximum de laisser des printf dans les codes définitif. Possibilité d'utiliser des macros:<br />#ifdef PRINT_TCH<br />#define PRINTF<img src="/redmine/plugin_assets/redmine_wiki_extensions/images/x_mark.png" alt="(x)"> printf(x);<br />#define <abbr title="x,y">PRINTF1</abbr> printf(x,y);<br />#define <abbr title="x,y,z">PRINTF2</abbr> printf(x,y,z);<br />#else<br />#define PRINTF<img src="/redmine/plugin_assets/redmine_wiki_extensions/images/x_mark.png" alt="(x)"> ;<br />#define <abbr title="x,y">PRINTF1</abbr> ;<br />#define <abbr title="x,y,z">PRINTF2</abbr> ;<br />#endif</p>
<p>Si la constante PRINT_TCH n'est pas définie, aucun message n'apparaît.</p>
<p>Autre point important: finir les messages par "\n"</p> LFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12032015-06-22T13:41:47Zthomas chust
<ul><li><strong>Assignee</strong> changed from <i>thomas chust</i> to <i>paul leroy</i></li></ul><p>a priori ces remarques ont été pris en compte dans la version 2.1:</p>
<p>[[<a class="external" href="https://hephaistos.lpp.polytechnique.fr/rhodecode/HG_REPOSITORIES/LPP/INSTRUMENTATION/USERS/CHUST/LFR_basic-parameters">https://hephaistos.lpp.polytechnique.fr/rhodecode/HG_REPOSITORIES/LPP/INSTRUMENTATION/USERS/CHUST/LFR_basic-parameters</a> ]]</p> LFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12052015-06-23T05:51:35Zpaul leroy
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Feedback</i></li><li><strong>Assignee</strong> changed from <i>paul leroy</i> to <i>thomas chust</i></li></ul><p>Il faudrait inclure basic_parameters.h dans basic_parameters.c sinon ça fait des warning à la compilation.</p>
<p>#include <basic_parameters.h></p> LFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12062015-06-23T08:48:31Zthomas chust
<ul><li><strong>Assignee</strong> changed from <i>thomas chust</i> to <i>paul leroy</i></li></ul><p>J'ai fait un nouveau push => version 2.2</p>
<p>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<br />void init_k_coefficients_f0( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );<br />void init_k_coefficients_f1( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );<br />void init_k_coefficients_f2( float *k_coeff_intercalib, unsigned char nb_binscompressed_matrix );<br />dans le .h</p>
<p>D'accord ?</p> LFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12072015-06-23T09:14:46Zpaul leroy
<ul><li><strong>Assignee</strong> changed from <i>paul leroy</i> to <i>thomas chust</i></li></ul><p>Ca marche si la fonction n'est pas destinée à être utilisée ailleurs que dans ton module basic_parameters.c.</p>
<p>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.</p> LFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12092015-06-23T16:06:00Zthomas chust
<ul></ul><p>Entendu: push fait ...</p> LFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12102015-06-23T16:08:42Zthomas chust
<ul><li><strong>Assignee</strong> changed from <i>thomas chust</i> to <i>paul leroy</i></li></ul> LFR-FSW - Task #446: BP1 BP2 pour release 3https://hephaistos.lpp.polytechnique.fr/redmine/issues/446?journal_id=12632015-07-02T13:08:24Zpaul leroy
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Closed</i></li></ul>