stillwater

The code is an edited version of the plussz max external from the SDK. It worked perfectly fine until I changed the number of outputs from 1 to 4 and added some math before the outputs were written. When a int or float is inputted it crashes Max/MSP entirely. Would anyone mind taking a quick look to see whats wrong? I am sure it is something basic but I can't see to find the error.

File is attached (OSX 10.5, Max/MSP 5, XCode 3)

The code is an edited version of the plussz max external from the SDK.  It worked perfectly fine until I changed the number of outputs from 1 to 4 and added some math before the outputs were written.  When a int or float is inputted it crashes Max/MSP entirely.  Would anyone mind taking a quick look to see whats wrong?  I am sure it is something basic but I can't see to find the error.  

max-crash-on-data-input

Make sure that you declare the inlets and outlets in the correct order. Also you might want to take a look to the type of your variable, you stores long in your structure, calculate with double and output double that are automatically cast to long.

Thanks for the reply. I declared the outputs in the correct order according to how things normally work in Max. Changed all of the floats to longs.

It doesn't crash anymore but it's not pulling the numbers in correctly. For example the 4th outlet should display the data inputted by the 4th inlet but the 4th outlet only displays the initial value displayed.

Thanks for the reply.  I declared the outputs in the correct order according to how things normally work in Max.  Changed all of the floats to longs.

It doesn't crash anymore but it's not pulling the numbers in correctly.  For example the 4th outlet should display the data inputted by the 4th inlet but the 4th outlet only displays the initial value displayed.


class_addmethod(c, (method)plussz_in3,      "in3",      A_LONG, 0); // the method for an int in the right inlet (inlet 3)


- in your int method, you don't assign the value to x->p_value0

b = ((x->p_value2/x->p_value3)*(x->p_value1/x->p_value3));


so in your tests it's likely that b will b equal to 0, at least that's the case with the default value (10/1000 = 0).

- in your int method, you don't assign the value to x->p_value0
- you're doing some integer division:

so in your tests it's likely that b will b equal to 0, at least that's the case with the default value (10/1000 = 0).


Awesome thank you so much for your help. Got the thing up and running this afternoon!

Awesome thank you so much for your help.  Got the thing up and running this afternoon!


Max crash on Data input