LAF repaint not being triggered on property change

d.healey

Changing a property of a control should repaint the control (at least that's what I thought) but it doesn't appear to be working properly, for tables anyway, even using the new sendRepaintMessage(); function.

HiseSnippet 958.3ocsV80aaaCDmzIrn1asXEXe.H7SJ.tF1ccsEqXqN1IdvXIod0YE6sBFJZKtPQJPQkLis.zm5a6659FrdTR1RIwHs0.SOHq6++ti2czSsFtHM0XQ3lmtLQfveMY1RsKZTDSpQSN.geH4XVpSXoErFtLgklJBQX7N+rmAt4tn7m+8UCYJllKpXgPu0H4hijwRWE2oC9EoRMlEJNUFWS6mNXB2nGYTlL.O6P5gRX7yYKDmv7p0ffv26vPoyXm4XNQJnyPS3xYQlK0E5+VYp7LkvSzGMCbTAaznHoJb5pbMEgv6NsJy2oHy+VxwxP4Z9UUfuIW.sxh50.bi6BR8+BfDtFj1s.ROhLiakItJId77UjIZ3.YNCJ00gRgtnF+SCxHCng10MlctXrEHVaQvy50qCEds2Ka0BJ2oNphMm9izUlvsBHSNxvYpiLly2WGNVHTAUpeAyROkA4T+ZVsP3FYhSLZfHncg31fMEe0MU3tkGgv5cJ7SWqXgz2iMNSycRiNncnkcYtsCgVfEVSlNrcG57UxWzgZN6O1q0e0pIcQ24P6z9JUPQqSZ2Kktn8UIQr0brhvbK5JzdmFReEsW2WP+A38S1CPwUdjH0JoVrNHTidXlyYz88Io0nB3qxvNPQPkIxieUFFztz6sWI+k0DqCeiHAZobGCmjPSMTQostBB6lKgkgt8ddOWBfQLk5LndDbSjAApFpOw3DuVGjitVW0hdSQymuQYk9RIraTreV0dWFFnyhOSXqWa7JBMyWeB4dedSH7hxRMEM5IZo60IhR5wFUnuy2+8smmPk0U+jUIBAn3xmqdP4bU9gCRB939jhCJTN3quAKj4X4KeP+1jC7eW5Ov0P3RDVmzmU3CDW.q5JlaaRNPjdtyj.Ft9LEx7OYz+ypH+SCVtl38+5fKkgtHDlf+.FghDxEQNOUGbdldyP7vxPTzljGiljxdl5AYRsfP+6AorKDSzSsBnmyuQone1u86HAyp8rP2dKErezDloXtquzze6Po.nO4Zap7aizoR2x52d7ErIs2ctI8yEhOhLU53QaFiM1.Fgi9+OvX48OOfb374BtqBf6RF+6a6kMehv+FSlSpWbLyYkP+.4jr3YvlRt.htVKTo9lqF9dqB5ddZeEXFrHKm3+fmRg88z3Rg8WIDEy3Vy63Eyd9a3teNG.S57KzaB+yBfltdniP50sGJFZ2dGm6S+GCC1a1lmrE17cagMOcKr462Bad1VXyy2BadwcZieRe+LmItXb.XL8v7cdX7gEWskuB3i6loziB

Christoph Hart

There is no obj.enabled property, so the condition is always false.

HiseSnippet 1011.3ocsV8taaaCDmxIrq1asXEXO.B9SJXdB1osYsqXqN1IdvX1Id0YE6asLRTVbghTfhJoFaAXuK6EYOR6MX6nnrkbqa2fwp9fft++6Nd2QMSICnYYRExo4EKSoHmOCOeoPGOLlvDnwmfbtOdJISSUtVVCVlRxxngHGm89dCCml6iJd9qmOfvIh.ZEKD5kRV.cBKgoq3Nq+Ov37QjP5ErjZZ+n9iCjhgRtLGvyd3tnTRvUjEzyHF0ZfQN24zPlVplqIZZFxY+Axvkyik2Hr5+RVF6RN0PzCMGbjk8HIOzfXCWzvXFOb1p7NCAdYVUUXOaU3KvSYgr07qpFedg.2JKpWObZrI71aC30qN75VCdaARN0fz9VH8.77.EKUWIwfmOEOV.GNQDnrWGJVcQM9iF3gRPCg1OgbEcjBHVag2Qc61wEdcvyZ0BJ8YZWNIx8acWYRfhBYxDY.gOQJu5XQ3HJk6Uo90Dk6EDHm5UypET8PYRpT.DdsshaC1X+xOipeGOBg03TEcgqT.tJhvyn.CfuOvjYZ.GkKBzLovqcnhbSgyF.8GKTxbQX6NtQqjuniq7xe4fV+ZqltK7ifdsi4bOaeUl+MLc7w7zXxZNJZXGSXetaW+m39Mv6CO.PysFDwDblft12fZCx0ZonmIYURtWvpLsCTL34zhvVjCEjfKZVk2dsoBCgAtVseVMwhvWPSgFM8T37EZ6g5raqaAOr8BaIPZefwykvYHgyuDJJduMNg.UKGNSpomK7JvZqaa491hhh1prRewopsJ1LMq9PF5IxStjppWoLJBs3aN2fe+yM0GqCrkkZJJEiEL84oTw6aXGUVKMyXknBButXB6dkSXEGHHFLGcWr8vAU.3560BIZBx4w3Gdj+lOWe9Pw.wS8mtbxgCKngWQeoQzSszHzOM9Di4kP.PCfvTpRyLIuyIzqgcl1g9l3SnYWokov5u0G8HmO4eEvuoBreW+kqI98er+MrPc7ZFu9M8iorEwUqm+SU+xlz54KpVzuiM52uL51Fshv2DW10UO9iqEe2eqeF4Z5XwLEE5ZMaprAyrUcBknDFVn2c6Gr2UFlyI5MWFatApT.zoswFPyVNQFSur9MT+usg9+JDe.dFSGDucL1XKXD5J9Xfwx60tG9znHZftBf6iG8yebtDC8BYtlIVLknULne.eVdxbXga.EhtPP4lIcmFldKKcWCsoBLGVEVP72vSovdFZmRg8VIDkPBTxWEXmjM2bd2BN.lDE+zPS3uW.Z20ivXbW+tnDnc6UAAlz+qf0Da2lC2Aad3NXyi1Aad7NXyQ6fMe8NXyS9f1XlzONWKSriC.iYmVrNzw4zU6cfU.+CfRvj9S

This works (with the sendRepaintMessage().

d.healey

@Christoph-Hart Oh, you haven't merged it yet - https://github.com/christophhart/HISE/pull/320 :)

Christoph Hart

@d-healey merged and fixed. The problem was that the repaint message was handled before than the property change so it didn't reflect the actual value.

d.healey

@Christoph-Hart Excellent, thank you!

d.healey

Got a couple of build errors:

../../../../../HISE/hi_snex/snex_library/snex_jit_NodeLibrary.cpp:154:60: error: cannot convert ‘snex::jit::Compiler’ to ‘snex::jit::Compiler*’
  154 |                 auto f = ScriptnodeCallbacks::getPrototype(c, ScriptnodeCallbacks::SetExternalDataFunction, numChannels);
      |                                                            ^
      |                                                            |

../../../../../HISE/hi_snex/snex_library/snex_jit_NodeLibrary.cpp:162:60: error: cannot convert ‘snex::jit::Compiler’ to ‘snex::jit::Compiler*’
  162 |                 auto f = ScriptnodeCallbacks::getPrototype(c, ScriptnodeCallbacks::HandleModulation, numChannels);
      |                                                            ^
      |                                                            |
      |                                                            snex::jit::Compiler

Christoph Hart

@d-healey Weird, they don't show up both on Windows and macOS (normally macOS complains at stuff like this)...

I fixed those two, but if there are more like this, you can fix it pretty easily by dereferencing (prepend a & before c).

d.healey

@Christoph-Hart Working perfectly now, thanks again!

LAF repaint not being triggered on property change

18

1.8k

10.4k

90.8k