Print Page - Convoy length in tiles.

Title: Convoy length in tiles.
Post by: inkelyad on 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_assembler.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?

Title: Re: Convoy length in tiles.
Post by: Bernd Gabriel on August 15, 2010, 05:25:31 PM

Thanks for spotting out this source of inconsistency.

I changed the code for convoy_summary_t.

Title: Re: Convoy length in tiles.
Post by: inkelyad on September 08, 2010, 04:12:52 PM

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

Title: Re: Convoy length in tiles.
Post by: Bernd Gabriel on September 08, 2010, 04:23:11 PM

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

Title: Re: Convoy length in tiles.
Post by: jamespetts on 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.

Title: Re: Convoy length in tiles.
Post by: inkelyad on September 13, 2010, 03:38:29 PM

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

Title: Re: Convoy length in tiles.
Post by: jamespetts on September 13, 2010, 11:34:40 PM

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

The International Simutrans Forum

Simutrans Extended => Simutrans-Extended development => Topic started by: inkelyad on August 15, 2010, 10:35:58 AM