[Fixed]1.9.8 config compatibility

[Kilinich]

Edit: (by tatt61880 on 2011.08.21 08:05)
Fixed in v1.9.9b. Thanks
-Tatt
----
looks like few "demo" functions removed from config and scenes which is using it stops working:

Code: Select all

spawnBoxRow = (offset, n)=>{for(n, (x)=>{Scene.addBox({pos = offset + [x, 0]})})};
spawnRing = {Scene.addPolygon({surfaces = [makeRing(1, 48), makeRing(0.5, 48)]})};
makeRing = (r, n)=>{
    list = [];
    for(n, (i)=>{
        a = (2 * math.pi * i) / n;
        list = list ++ [r * [cos(a), sin(a)]]
    });
    list
};

Please get it back...

[tatt61880]

I've never met the scenes which use these functions.
These functions were removed at v1.9.5 (2011.06.1), as far as I know.

This is minor issue and keeping config simple will be better, in my humble opinion.

[electronicboy]

I've never met the scenes which use these functions.
These functions were removed at v1.9.5 (2011.06.1), as far as I know.

This is minor issue and keeping config simple will be better, in my humble opinion.

The only problem is that scenes that DO use these functions, will be broken, which needs to be avoided!.

[tatt61880]

I've never met the scenes which use these functions.
These functions were removed at v1.9.5 (2011.06.1), as far as I know.

This is minor issue and keeping config simple will be better, in my humble opinion.

The only problem is that scenes that DO use these functions, will be broken, which needs to be avoided!.

As I wrote, there are not so much scenes which use these functions and "keeping config simple" is better than keeping minor function, at least, the function shouldn't be in [root], in my humble opinion.

[TC42]

I would keep those functions, just because they were there in previous versions, and because some people will come looking for them eventually. I really don't care how complex the config is, as long as it works properly. But then again, I'm not a dev.

/posted using in-game browser

[tatt61880]

I see.
I won't deny your opinions about this topic, anymore.

Edit:
Sorry for selfish opinion.

[Kilinich]

It's not only useful functions but great example of thyme programming for beginners. I vote for keeping it.

[Someone Else]

Even though I've never used it, and I was already pretty well acquainted with Thyme when I found about it, I agree with Kilinich and TC42.

[emilk]

I'll re-add them in next beta