Macro: length as param name not allowed

[andreasplesch]

While making a new arrow macro, I wanted to use 'length' as param name for the length of the desired arrow:
http://andreasplesch.github.io/x3dv4/test/Macro.html
However, this did not work, and I need to work around with the name 'lengthness'. The reason is that length is a reserved word for array indices, see below.
There is an easy solution. Just make params an object instead of an array, in line 267:
var params = {} ;
I did not test but this may be the only change necessary.

-- arrays vs. objects

aa = [1, 2];
Array [ 1, 2 ]
aa.length
2
aa['length']
2
bb = {};
Object { }
bb['i1']=12
12
bb['i2']=13
13
bb['length']
undefined

[DRx3D]

Andreas,

Hmm, that would be bad. I will add this to the fixes. It doesn't look
like it impacts anything external so the work should go easily.

Leonard Daly

While making a new arrow macro, I wanted to use 'length' as param name
for the length of the desired arrow:
http://andreasplesch.github.io/x3dv4/test/Macro.html
However, this did not work, and I need to work around with the name
'lengthness'. The reason is that length is a reserved word for array
indices, see below.
There is an easy solution. Just make params an object instead of an
array, in line 267:
var params = {} ;
I did not test but this may be the only change necessary.

-- arrays vs. objects

aa = [1, 2];
Array [ 1, 2 ]
aa.length
2
aa['length']
2
bb = {};
Object { }
bb['i1']=12
12
bb['i2']=13
13
bb['length']
undefined

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#4, or mute the thread
https://github.com/notifications/unsubscribe-auth/AH4P5w0CRKUuy-Zy-cR0RLjkZei-rQWdks5qlC3dgaJpZM4JwhCh.

Leonard Daly
3D Systems & Cloud Consultant
X3D Co-Chair on Sabbatical
LA ACM SIGGRAPH Chair
President, Daly Realism - /Creating the Future/

[andreasplesch]

Yes, it should be straightforward.

On Sep 2, 2016 12:24 AM, "DrX3D" [email protected] wrote:

Andreas,

Hmm, that would be bad. I will add this to the fixes. It doesn't look
like it impacts anything external so the work should go easily.

Leonard Daly

While making a new arrow macro, I wanted to use 'length' as param name
for the length of the desired arrow:
http://andreasplesch.github.io/x3dv4/test/Macro.html
However, this did not work, and I need to work around with the name
'lengthness'. The reason is that length is a reserved word for array
indices, see below.
There is an easy solution. Just make params an object instead of an
array, in line 267:
var params = {} ;
I did not test but this may be the only change necessary.

-- arrays vs. objects

aa = [1, 2];
Array [ 1, 2 ]
aa.length
2
aa['length']
2
bb = {};
Object { }
bb['i1']=12
12
bb['i2']=13
13
bb['length']
undefined

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#4, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AH4P5w0CRKUuy-Zy-
cR0RLjkZei-rQWdks5qlC3dgaJpZM4JwhCh>.

Leonard Daly
3D Systems & Cloud Consultant
X3D Co-Chair on Sabbatical
LA ACM SIGGRAPH Chair
President, Daly Realism - /Creating the Future/

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#4 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF4p62nqihB2zyVJbBzn-b38HNzawBDSks5ql6T6gaJpZM4JwhCh
.