WonderlandEngine / api

Wonderland Engine's JavaScript API.
MIT License
10 stars 6 forks source link

Add getter with out params #16

Closed signorpipo closed 9 months ago

signorpipo commented 10 months ago

This left out materials getter and also index data getters which are a bit more complex

DavidPeicho commented 9 months ago

Please modify all original getters to re-use getX(), so that we don't duplicate code especially if we have an issue somewhere.

DavidPeicho commented 9 months ago

@SignorPipo I shouldn't have asked you to modify get/set extent actually, for backward compatibility, we need them exactly like before without using getExtents. People might be using them like that:

component.extents[0] = 1;

And that's not going to work with getExtents

DavidPeicho commented 9 months ago

Merci for this contribution!