-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Converting to Arduino Lib #3
Comments
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 <#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> .
|
Je connais... prend tout ton temps! ;-)
J'ai essayé, mais sans succès.
genre ?
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
The text was updated successfully, but these errors were encountered: