[9.9] bug? spacing/scheduling does not seem to save

[ӔO]

the spacing/cnv month always seems to revert back to 1 no matter what settings are used in that dialogue box.

[Carl]

Yep, I'm getting this too.

[jamespetts]

Thank you for reporting this. Inkelyad - are you able to look into this? AEO - do other changes in that dialogue box (such as max. wait for load and alternate directions) seem to save? Sorry that you are having trouble.

[inkelyad]

It is problem i reported in neighbour topic.

in wkz_change_line_t::init

Code Select Expand


sscanf( p, "%c,%hi,%hi", &tool, &line_id, &livery_scheme_index );


You always read livery_scheme_index from tools param.

But, for example, 'g' command don't put it there.
line_management_gui_t::infowin_event(const event_t *ev):

Code Select Expand


werkzeug_t *w = create_tool( WKZ_LINE_TOOL | SIMPLE_TOOL );
cbuffer_t buf;
buf.printf( "g,%i,", line.get_id() );
fpl->sprintf_schedule( buf );
w->set_default_param(buf);


[jamespetts]

Ahh - apologies. I did not understand the significance of this point. I shall have to look into that. Thank you for explaining.

[ӔO]

@james
the other settings seem to save, but the spacing does not.

[jamespetts]

I have now looked into the matter (thank you, Inkelyad, for finding the problem), and a fix has been applied on my 9.x branch.

[inkelyad]

I can be dense, but why not like this?

(Start of wkz_change_convoi_t::init same as in Standard, and livery_scheme_index is readed in 'V' case)

[jamespetts]

Inkelyad,

thank you very much for that - that seems a lot neater! Incorporated into the 9.x branch.

[jamespetts]

9.10 with this fix incorporated has been released.