Skip to content
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.

Sign up for GitHub

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

Jump to bottom

supernova: setting an array onto a control that doesn't exist will set value that does exist #916

Closed
miguel-negrao opened this issue Jul 18, 2013 · 4 comments
Closed

supernova: setting an array onto a control that doesn't exist will set value that does exist #916

miguel-negrao opened this issue Jul 18, 2013 · 4 comments
Labels
bug Issues that relate to unexpected/unwanted behavior. Don't use for PRs. comp: supernova

Comments

@miguel-negrao
Copy link
Member 

SynthDef( "test2", { |b = 1|
Out.ar( 0, Silent.ar );
}).add;

z = Synth("test2")

z.set(\a, [2,2])

scsynth:

NODE TREE Group 0
   1 group
      1000 test2
        b: 1

supernova:

gives warning

argument number out of range

and sets b to 2

NODE TREE Group 0
   0 group
      1 group
         1000 test2
            b: 2

This breaks ULib's UGloabalEQ which depends on setting an array to the rootnode to update all synths that have the eq controlname.
@miguel-negrao miguel-negrao mentioned this issue Jul 18, 2013
Closed
@miguel-negrao
Copy link
Member Author

Wow, 14 minutes from bug report to fix... this is one of the reason I love open-source software !! :-D

@timblechmann
Copy link
Contributor

much more fun to fix my own bugs than other people's ... and you helped a great deal by providing a reproducer!

@miguel-negrao
Copy link
Member Author

Btw, it still overflows if you set an array bigger then the size declared for the control. The behaviour is the same in scsynth, so I suppose it is known/documented ?

SynthDef( "test2", { |a = #[1,1], b = 1|
Out.ar( 0, Silent.ar );
}).add;

z = Synth("test2")

z.set(\a, [2,2,2])

@timblechmann
Copy link
Contributor

yes, not sure, if it is a bug/feature by design or by laziness.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Issues that relate to unexpected/unwanted behavior. Don't use for PRs. comp: supernova
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@miguel-negrao @timblechmann