Open soif opened 3 years ago
Hello,
Okay, merci. je vais voir ça quand j’ai un peu de temps (j’ai un taf pro de ouf)
Souvent on règle ça en imposant de définir la macro AVANT d’inclure la lib.
Il serait bien dans ce cas de changer l’option de compile pour forcer le dev à la définir avec une valeur.
A++
@.***
Philippe Demerliac
Visitez mon site http://www.cyrob.org http://www.cyrob.org
Et regardez ma chaine YouTube : https://www.youtube.com/c/PhilippeDemerliac_Cyrob https://www.youtube.com/c/PhilippeDemerliac_Cyrob
Demandez à rejoindre le groupe Facebook : Cyrob.org
Le groupe discord Cyrob : https://discord.gg/mNMSE53 https://discord.gg/mNMSE53
Soutenir Cyrob.org : http://philippe.demerliac.free.fr/Support.htm http://philippe.demerliac.free.fr/Support.htm
De : soif @.> Envoyé : lundi 13 septembre 2021 15:57 À : Phildem/Cath @.> Cc : Subscribed @.***> Objet : [Phildem/Cath] Converting to Arduino Lib (#3)
J'ai converti ton code en une version Librairie Arduino Standard. :-p
Tu peux l'essayer dans ma Branche "ArduinoLib", à : https://github.com/soif/Cath/tree/ArduinoLib Le "remote repo" a rajouter: https://github.com/soif/Cath.git
Je n'ai pas testé sur le hardware, mais ca compile bien, sur Arduino IDE + PlatformIO
Si ca te branche de merger ça, je te fairais un PR (à ta demande), une fois que tu aurais validé que ca marche toujours bien sur le hardware...
PS: Pour finaliser, ensuite, il faudrait certainement que les 2 defines (CATH_MAX_TASKS et CATH_SMALL_COUNTER) puissent etre définis (optionellement) dans le sketch principal, et non plus seulement au niveau de la librairie.... J'ai eu beau essayé, ça dépasse mes maigres aptitudes en C++ & directives de compilateur.... Si ca te dis...
HTH
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Phildem/Cath/issues/3 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRB3LAPFTBFNJXUYRSHKJ3UBX7KXANCNFSM5D54NYLA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
Okay, merci. je vais voir ça quand j’ai un peu de temps (j’ai un taf pro de ouf)
Je connais... prend tout ton temps! ;-)
Souvent on règle ça en imposant de définir la macro AVANT d’inclure la lib.
J'ai essayé, mais sans succès.
Il serait bien dans ce cas de changer l’option de compile pour forcer le dev à la définir avec une valeur.
genre ?
#ifndef CATH_MAX_TASKS
#halt_compil (je connais pas la syntaxe)
#endif
J'ai plutot tenté de definir une valeur par default , si non renseignée (avec un warning de compil qui l'affiche)... mais j'ai échoué lamentablement... J'suis pas du tout un as du C++...
Je te passe la main.. quand tu aura le temps.
a+
J'ai converti ton code en une version Librairie Arduino Standard. :-p
Tu peux l'essayer dans ma Branche "ArduinoLib", à : https://github.com/soif/Cath/tree/ArduinoLib Le "remote repo" a rajouter: https://github.com/soif/Cath.git
Je n'ai pas testé sur le hardware, mais ca compile bien, sur Arduino IDE + PlatformIO
Si ca te branche de merger ça, je te fairais un PR (à ta demande), une fois que tu aurais validé que ca marche toujours bien sur le hardware...
PS: Pour finaliser, ensuite, il faudrait certainement que les 2 defines (CATH_MAX_TASKS et CATH_SMALL_COUNTER) puissent etre définis (optionellement) dans le sketch principal, et non plus seulement au niveau de la librairie.... J'ai eu beau essayé, ça dépasse mes maigres aptitudes en C++ & directives de compilateur.... Si ca te dis...
HTH