Bug #1255

jcom.parameter of type array are limited to 255 items

Added by Julien Rabin over 6 years ago. Updated over 6 years ago.

Status:ClosedStart date:2012-12-03
Priority:NormalDue date:
Assignee:Théo de la Hogue% Done:

0%

Category:-Spent time:-
Target version:MVC for 0.6
Branch: OS:

Description

This was discussed on the list some time ago but not reported in redmine as it seems. With a parameter of type array, max number of items is 255. Sending a list longer than 255 makes Max (6.0.8) freeze forever (well, at least 30~ minutes).

It would be great to have a max lenght much longer than this.

History

#1 Updated by Nils Peters over 6 years ago

AFAIR, I fixed this for Jamoma 0.5x
There are even tests for that.

#2 Updated by Théo de la Hogue over 6 years ago

  • Status changed from New to Resolved
  • Assignee changed from Théo de la Hogue to Julien Rabin

actually this not seems to be a jamoma problem but a zl attribute to set (@zlmaxsize).
the path below works for me for list of 1024 items.


----------begin_max5_patcher----------
544.3oc2V1rabBCDG+L7TX4yTD1KvB8T5yQUUjAb23UfMxXR2rQ4cu1FX+Ra
AR5VRRu.xiGl4+7yiM9YWGXlXGsAB9J36.GmmccbrlLFb5G6.qH6xKIMV2fY
sJkfC85lh2VIZUkTkcRTu0NSpmpocQFlQ3affezOcMQk+.iu4dIMW04ANxOv
CfVYdtZ8vyCeAqvlaQ11ufiNI2L9gTar8hqq4g2LKEc.xnxgv8SAW0v1akLB
qkv0pP7epBYbEzaFEpMx.bPp4UXR2fqWoImpLNoxlJ32jLRI7PBjZ6Jp7dJm
jUZ8H3lgG5uzxXHUJ5NaI.2WBJobvc6K0da.F.EfCuMTzfvYfOTTne5pfzfD
SOy5wfX5zP7LPg+jzGEO+9nvOl8QayEU9GRLPQaTf6Lj.PjRxSKTCUGGQXaS
DJIZLPhdk8R+uroqmQI34toKdQ10ccR0tm85YypQ+wk2w2lsmS+erHKchhGi
QQIueLR2MsQJZqsf5csoJIrq0BMJpv+EnxFPXIie40brZxX+b90HZk4CUwP1
AG0Ug9bJFmnX5KAczIzY97.qnfxO8PzBVi4bUaMEb0ExapbRmPNUrhZgtWtG
IqQ9Q5UATjeX+Fb7EVNluaWgn2..vSTHWTs+KAq4hkSBVqnWF8DmNC8DtXxA
MC0DmtXxYNv4stV0cjAot9QproOjVgnObcqPZFF6YGx3cCsQDJoOxF7Ow0Ds
Wb+Mm6zEVB
-----------end_max5_patcher-----------

#3 Updated by Julien Rabin over 6 years ago

  • Assignee changed from Julien Rabin to Théo de la Hogue

Thanks for looking into this. Actually, after further investigation, the problem comes from jcom.remote and not from jcom.parameter. See the attached crashTest patch.


----------begin_max5_patcher----------
1447.3oc6YtzaiaCD.9r8uBVgBrGpqWIpGVpmZu0Bzsn.83hh.ZIFKtQu.Ic
drK1+6kOjjkrkjYx53j3s4fC7vWC+FNyPN9KymYst7dLyB7KfOBlM6KymMSI
RJXV82mYkitONCwTcyJtLOGWvsVnaiiumqj+O3hDRwF.B.8C9I.giyYfLBiC
xQ2fAe.cO3ZJF+YbyHyHE33xsEpgCqEVg3woho4JJNlqUq.G6k1K.qbje53F
J+mq6Rav+VOFRhRAJW+oe1A1L6WS2rVJVNVsjhs4jhLLWsMbZ5VYAu.kiUyv
uQInLqccubKuo+1c5+0nX79yAi7YsL3RaozuNet7iEeiP8uJAUzx0Y3bPJlh
WZMNlfQq5fI+H4+f1CioUWVT5Ow72w.avb.99XwAujkKmhTAAqV5J9ywU.Ju
vk9RP4NEubOC75YhM+NJEAJJ4.VLhhSDtmrswox+kixxzNn+vTvxGJAjiSjA
bx6sKm9i2kCVWJIzcDdpfK2J3vRvemgQLrNDVNFDSQrT.krIkCHEfGJ2RAxC
56PXunZtSv0Hsi5JeEd88jeya0vbM30OWywLFZC9.tFZaczP60Gtb0mzBGIz
tu0PaY3ibK2fHsH9CUXspXY0tnOizPjZz.eM8YhififKBb3OENbUAab7L4zw
pK.bHmVvO5bbGFXfuJbgJI+nHI7sDR1lwIrLRBlZv1ORmEJLPs+gicYvncSE
UrS4X5U3Bj31Tci8wvbAcDJidIzAjr6lfavPr8gTCTanPn8nTaghbOADUfuS
rqN3PymDYzVRw4kbL384O7gxj2eWJhySwIaiu43vzKPk5wIZxL6PaqSQBGuI
wxhV77BmWp9tzlkW5hOsjqajwokht3yJ4BsMOqj82EYkpcWLKqjyEWVolsuY
Ykfe2jThgKR1Kkz0aKt43jrIkzpIKhgi2IIkzKqyUGt.b.Pf6DzwwWmUJPe.
y1eR+L22P9YibJpBnN7bbj.gppBFXO04kvWpiKcUeQfDMFp4fn2jL7sXJiTV
zY9mYgpp5HdVmgHg2mJUSTvhVQjBsH6VQT7sjlwGpEJLIyaZrgg5JOn837zf
LJnCCEFuMYkw2fS5N8VkU3BRQEEKby4Hd85z1bB9ZjH54U8fgyxAauoTgC13
fFkYVanjjxBoRzajRwMKm73gtXnc2LpdTfpFXvLw9XKaMhJ4VcDYXSi7xxr9
M0ZpDGtQEjbDGyIZkEZ2Noj7JJQWNnVY5.9orXZYVVuoR2xsCzRhvZFiuijv
S6UAHYKhtSpZLAVsLJgrAy38kwQaX8kz6W.n6wrtds8jOYrswbUCqMFqBj2X
xqt7WchdMZENFMJ1zNti67dDG3QipMqIxVsqzSjW8h1MPdy1KF.57PNvuJUU
.hRQOXHog5zmtG7vtQKdxngGezjFNIoWbFosd0OBxbBf5GAqBCB8a9bLh4YD
vFrXqpVZpBsxQ7DtYU5gY60Hay1ptitUsOE9YOWj3XNY4kI3LCOU3o7fBz29
JZB+nWUtQNmT2HyBx6ZG93Bx++w369B.yiw2.5lX7NGKFO7MPH9cWRS9qnMx
cRTvP19vVAV4VZbiVzjbCzGKhKLwIEs2Y8i6tug3MXc6XJIIo+cD0W0NopTb
ktZELD5opAjHXAzWYRB1889qKgIucm9dzFcX6wtYcMcudP+Fdq97qwPC03mn
ww0YUWiypvnWLaiig6TuSjsQWOm1m7YwP2hStRrBhfHWIh4PIq2x0dYceO52
5CyN9C+dDuVYPkXhEerEU7.10nr5Wf2N1AdTz7cPrtFFJKQ+.R5RZru8utvF
GZ2i5ZPGwnK+8U5zoCr3GZsOnDKFqOAFnOxe7vyk93+JSeV8JSeBMQehNa5C
zD0w67gGGSbuNepisI3w47oOutTmnWYzw0.84o5pqybrWcbkZxd0ucuZ2tec
aEy1Wm+e.Go4pDB
-----------end_max5_patcher-----------

#4 Updated by Julien Rabin over 6 years ago

  • Status changed from Resolved to Assigned

Do you want another ticket for this or is this one fine ?

#5 Updated by Théo de la Hogue over 6 years ago

  • Status changed from Assigned to Resolved

Thanks for the test patch.
This error was in a function that iterate to copy an atom but the index were a TTUInt8 (0 255) instead of a TTUInt32 (0 to many many)

It is fixed now but you have to checkout the Core repository because the method was in a file which is in the Shared folder.

#6 Updated by Julien Rabin over 6 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF