Convoy length in tiles.

Topic: Convoy length in tiles. (Read 3610 times) previous topic - next topic

Convoy length in tiles.

August 15, 2010, 10:35:58 am

From gui/convoi_detail_t.cc:

sprintf( tmp, "%s %i", translator::translate("Station tiles:"), cnv->get_tile_length() );

From gui/components/gui_convoy_****embler.cc

Code: [Select]

translator::translate("Station tiles:"), (vsum.length + TILE_STEPS - 1) / TILE_STEPS);

From: gui/components/gui_convoy_label.cc

Code: [Select]

translator::translate("Station tiles:"), (convoy.get_vehicle_summary().length + TILE_STEPS - 1)/TILE_STEPS);

As I can see get_tile_length is not '(vehicle_summary.length + TILE_STEPS - 1)/TILE_STEPS)'
What version is right?

Re: Convoy length in tiles.

Reply #1 – August 15, 2010, 05:25:31 pm

Thanks for spotting out this source of inconsistency.

I changed the code for convoy_summary_t.

Re: Convoy length in tiles.

Reply #2 – September 08, 2010, 04:12:52 pm

It is not merged to main simutrans-exp repository. Why?

Re: Convoy length in tiles.

Reply #3 – September 08, 2010, 04:23:11 pm

James did not merge anything to the main simutrans-exp. We have to wait...

Re: Convoy length in tiles.

Reply #4 – September 11, 2010, 08:41:51 pm

Sorry for being slow with the updating - I have been rather preoccupied recently, but am catching up now. I have merged all of the latest updates into the -devel branch.

Re: Convoy length in tiles.

Reply #5 – September 13, 2010, 03:38:29 pm

James, please merge this patch too (from Gabriel master brunch). It is very annoying bug.

Re: Convoy length in tiles.

Reply #6 – September 13, 2010, 11:34:40 pm

Ahh, I'd forgotten about this patch - thank you for reminding me! Merging now.