RawSignal.Multiply is 0 when doing plugin initialization, it must not be used to do divisions

cpainchaud / RFLink32

RFLink for ESP, with MQTT client

Other

119 stars 44 forks source link

RawSignal.Multiply is 0 when doing plugin initialization, it must not be used to do divisions #39

Closed obones closed 2 years ago

obones commented 3 years ago

This happens via PluginInitCall and so we must not use its value to perform "const" computation.

This leads to crashes when the image is built in debug mode.

cpainchaud commented 3 years ago

Bonjour @obones,

ne penses-tu pas qu'il faudrait plutot revoir le code en amont : y'a-t-il un intérêt à appeler PluginInitCall tout court? Je n'ai pas/peu identifié de code parculier d'initialisation de plugins, et toi ?

obones commented 3 years ago

Ah bah j'ai un gros doute sur son utilité effectivement, je n'ai pas vu de tel plugin non plus. Mais en l'état, je suis allé au moins intrusif sur le coeur de l'appli