Warning to overflow of convoi IDs March 22, 2009, 11:22:39 am I realised that there is no warning to the overflow of convoi IDs. When I tried to buy 8193rd convoi (under the default setting i.e. convoys = 8192), Simutrans just shut down. It would be good if there is a warning message, if you tried to buy 8193rd convoi... Quote Selected
Re: Warning to overflow of convoi IDs Reply #1 – March 22, 2009, 07:31:18 pm Quote from: yoshi – on March 22, 2009, 11:22:39 amSimutrans just shut down.So this is a bug => topic moved. Quote Selected
Re: Warning to overflow of convoi IDs Reply #2 – March 22, 2009, 09:33:14 pm A map with 8192 convoys...... holy. cow. EDIT: WAit.. wait... wait... A map with 8192 convoys - AND YOU STILL NEED MORE. Good grief! (I'm just amazed... hehehehe) Quote Selected
Re: Warning to overflow of convoi IDs Reply #3 – March 23, 2009, 02:22:36 pm well, mazbe the default should be 32760 anyway ...But a warning is possible. Same would go for just denz buzing a new one. Quote Selected
Re: Warning to overflow of convoi IDs Reply #4 – March 23, 2009, 02:36:43 pm What about this one with the following sentence.... Leave the convoys alone, now!!!Edit: sentence modified. Quote Selected Last Edit: March 23, 2009, 03:43:27 pm by vilvoh
Re: Warning to overflow of convoi IDs Reply #5 – March 23, 2009, 03:20:37 pm Vilvoh, just so that you know: that could be easily interpreted as insulting (at least I can imagine that instantly).How much memory does that value eat anyway? 8B per item when aligned? Quote Selected
Re: Warning to overflow of convoi IDs Reply #6 – March 23, 2009, 03:53:11 pm Quote from: VS – on March 23, 2009, 03:20:37 pmHow much memory does that value eat anyway? 8B per item when aligned?4, i think. It's just a table of pointers, which is used to implement the "tombstone" pattern to have checkable pointers (checkable if the object in question still exists).http://en.wikipedia.org/wiki/Tombstone_(programming)Edit:If someone ever wondered what "quickstones" are, that's been a modification of this pattern, which is supposed to be faster. No idea if this is still in the current Simutrans code, but it was when I left it. Quote Selected
Re: Warning to overflow of convoi IDs Reply #7 – March 23, 2009, 04:24:40 pm Quickstones are indeed used heavily in the current Simutrans code. Quote Selected
Re: Warning to overflow of convoi IDs Reply #8 – March 24, 2009, 12:18:27 pm THe new code will extend to 65530 and give a warning (and no new convoi) when handles are exceeded. Meaning the values will be gone from configuration files soon. Quote Selected
Re: Warning to overflow of convoi IDs Reply #9 – March 24, 2009, 12:40:49 pm Oh, thank you very much for your prompt work! Quote Selected