Commit bc08111a authored by Mathieu Giraud's avatar Mathieu Giraud

core/filter.{h,cpp}: move typedef in .h

parent 8b1b7c74
Pipeline #32495 passed with stages
in 39 minutes and 3 seconds
......@@ -98,7 +98,6 @@ BioReader FilterWithACAutomaton::filterBioReaderWithACAutomaton(
sorted map */
}else{
/* sort map */
typedef function<bool(pair<KmerAffect, int>, pair<KmerAffect, int>)> Comparator;
Comparator compFunctor = [](pair<KmerAffect, int> elem1 ,pair<KmerAffect, int> elem2){
return (elem1.second == elem2.second) ? elem1.first > elem2.first : elem1.second > elem2.second;
};
......
......@@ -3,6 +3,8 @@
#include "bioreader.hpp"
#include "automaton.hpp"
typedef function<bool(pair<KmerAffect, int>, pair<KmerAffect, int>)> Comparator;
class FilterWithACAutomaton {
private:
......
  • mentioned in issue #3361 (closed)

    Toggle commit list
  • Pourquoi sortir le typedef de la fonction ? On ne l'utilise qu'une seule fois.

  • C'est ce qui a permis que le job passe, y compris sous valgrind. Je ne suis pas sûr d'avoir tout compris l'erreur. En tout cas les typedef sont typiquement des choses de .h, on ne les inclut habituellement pas dans du code.

    cc @mikael-s

  • C'est ce qui a permis que le job passe, y compris sous valgrind

    En fait non. C'était juste un coup de chance : le premier job (qui a échoué) a tourné sous le nouveau slave qui a g++-7 par défaut. Le second job (qui a réussi) a tourné sous kapla qui a g++-5 par défaut.

    En tout cas les typedef sont typiquement des choses de .h, on ne les inclut habituellement pas dans du code.

    Là c'est une déclaration locale pour alléger la lecture associée à une fonction interne de ce type. Ce n'est pas choquant que ce soit dans la fonction.

  • En fait non. C'était juste un coup de chance

    ok

Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment