News:

SimuTranslator
Make Simutrans speak your language.

Convoy length in tiles.

Started by inkelyad, August 15, 2010, 10:35:58 AM

Previous topic - Next topic

0 Members and 2 Guests are viewing this topic.

inkelyad

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

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

From: gui/components/gui_convoy_label.cc

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?

Bernd Gabriel

Thanks for spotting out this source of inconsistency.

I changed the code for convoy_summary_t.
The journey is the reward!

inkelyad

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

Bernd Gabriel

James did not merge anything to the main simutrans-exp. We have to wait...
The journey is the reward!

jamespetts

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.
Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.

inkelyad

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

jamespetts

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

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.