Factory_x should have "tile" list instead of "coord"

[Yona-TYT]

I made this little patch so that "get_tile_list" gets an array with "tile_x": CHG-Get-tile-list-in-factory_x.patch

Code Select Expand

faxtory_x(pos.x, pos.y).get_tile_list()

[Andarix]

??

http://dwachs.github.io/simutrans-sqapi-doc/classfactory__x.html#a59a5d79eac6ae4618c34f382c616b4d3

[Yona-TYT]

??
http://dwachs.github.io/simutrans-sqapi-doc/classfactory__x.html#a59a5d79eac6ae4618c34f382c616b4d3

I guess it's a bug in the doc, but what it actually returns is a "coord" array not a "tile_x" array.

This patch fixes just that.

[Yona-TYT]

Test in action:

Code Select Expand

local fact = factory_x(pos.x, pos.y)
local list = fact.get_tile_list()
foreach(t in list) {
t.mark()
}

[Andarix]

I guess it's a bug in the doc, but what it actually returns is a "coord" array not a "tile_x" array.

This patch fixes just that.

And what's so bad about it if it's coord?

I'm afraid that this patch will demolish the script AI.

It would make more sense to make a function coord_to_tile_x for that.

[Yona-TYT]

And what's so bad about it if it's coord?

I wanted to be consistent with the other "get_tile_list()" functions, since factory_x is the only class that doesn't return a "tile_x" array.


 Maybe making an additional function to make coord list independent is a good idea, something like get_coord_list().

[Dwachs]

If tile_x would actually be derived from coord3d then changing get_tile_list should not bring any problems.

But tile_x is not derived from coord3d, actually, which is another bug in the docs. Will see, what can be done.

[Dwachs]

This is now in r10821. If there are any problems with existing scripts due to this change, please report back.

[Yona-TYT]

This is now in r10821. If there are any problems with existing scripts due to this change, please report back.

You don't know how much I appreciate this!. 

With this I am further optimizing the code of the basic class in the tutorial.

A lot of "coord" passed to "tile_x".