Mackensen wrote:Looking back over the posts in this topic, I think there are two reasons this patch wasn't maintained:
1) Its primary maintainer is inactive.
Then your and my opinion on inactive differ, although it is true that the last commit activity of that person was just under a week ago. So yes, he was inactive w.r.t. commits the last five days.
Mackensen wrote:2) The assumption was that the forthcoming new map array would force a rewrite anyway.
Get your assumptions right. The new map array project (i.e. map branch) stranded half a year before the introduction of this thread. There might have been other attempts at a new map array, but they started after the map accessor 'project' which started after this patch was not maintained anymore.
Mackensen wrote:The map array, IIRC, is in development hell now. A number of fundamental things have changed since the initial write, including the C to Cpp migration. I think YAPF was introduced after the fact, or around the same time. I'm playing with the source code now but I'm not that familiar with it. It's (almost certainly) not a quick fix, else someone would have done it by now.
The map array is not hampered by the C->C++ migration; after all, it was just compiling C with the C++ compiler at first. The map array is still very very C-like. Whether or not YAPF was introduced doesn't matter a tiny bit; the pathfinders work mostly via an abstraction layer thus they aren't the main problem. That it's not a quick fix anymore has more to do with the other changes to the code and the assumptions than those two projects you mentioned.
For example the "bridge over anything" feature has broken this patch far further than any of the other things. The introduction of tram tracks probably comes at a steady second place.
Note 1: that there has been a second attempt after the bridge over anything feature was introduced for only cbh for rail, but that stranded because trains broke up when turning around on the bridge; this might be related to signals, but that I'm not sure about.
Note 2: I was not around at the time of this patch, however the subversion history (i.e. the map branch) and a look at the patch tell me quite a lot. I started working on the second map rewrite attempt with a few others (LordOfThePigs, glx, egladil, Belugas) but then Tron took over and finished it in trunk.