search

Ixox / preenfm3

Official repo of the preenfm3 FM synth
107 stars 23 forks source link

note freq estimation should be > 0 #29

Closed pvig closed 1 year ago

pvig commented 1 year ago

Hello Xavier, voici un PR concernant un plantage que j'ai rencontré dans quelques uns de mes patchs, depuis un bon moment, je pensais avoir corrompu mes patch à force de bidouiller dans tout les sens. Mais non, j'ai finalement identifié le problème :

ça plante sur TftDisplay::oscilloRrefresh lorsque l'on va dans la partie éditeur, avec ces conditions : Si dans un patch, une porteuse, en mode OSC_FT_KEYHZ, a pour fréquence 0, et que le fine tune est réglé en négatif. alors dans TftDisplay::oscilloNewLowerFrequency, des valeurs négatives sont calculées => plantage.

Ixox commented 1 year ago

Hello Patrice, OK j'arrive a reproduire un plantage. Merci :) Mais pourquoi as-tu fermé cette PR ?

Ixox commented 1 year ago

OK i made this fix in the original main. Thanks again !