summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorrubidium <rubidium@openttd.org>2010-09-16 16:28:40 +0000
committerrubidium <rubidium@openttd.org>2010-09-16 16:28:40 +0000
commit9a0a753d9610e258b602c465ef02b703c1ad9457 (patch)
tree09d2d2bf6e767ae72ecdce5ed3cf47a662e77ebd
parent7a88433228faeb858e7e53a86a237c61e3d4eea1 (diff)
downloadopenttd-9a0a753d9610e258b602c465ef02b703c1ad9457.tar.xz
(svn r20814) -Codechange [FS#3835]: make waypoint default names work like depots, stations and vehicles (Krille)
-rw-r--r--src/saveload/saveload.cpp3
-rw-r--r--src/saveload/station_sl.cpp1
-rw-r--r--src/station_base.h1
-rw-r--r--src/station_cmd.cpp46
4 files changed, 28 insertions, 23 deletions
diff --git a/src/saveload/saveload.cpp b/src/saveload/saveload.cpp
index ba2178717..ce604d0a6 100644
--- a/src/saveload/saveload.cpp
+++ b/src/saveload/saveload.cpp
@@ -211,8 +211,9 @@
* 146 20446
* 147 20621
* 148 20659
+ * 149 TODO
*/
-extern const uint16 SAVEGAME_VERSION = 148; ///< current savegame version of OpenTTD
+extern const uint16 SAVEGAME_VERSION = 149; ///< current savegame version of OpenTTD
SavegameType _savegame_type; ///< type of savegame we are loading
diff --git a/src/saveload/station_sl.cpp b/src/saveload/station_sl.cpp
index 72d7c7ecf..dabb3e251 100644
--- a/src/saveload/station_sl.cpp
+++ b/src/saveload/station_sl.cpp
@@ -238,6 +238,7 @@ const SaveLoad *GetGoodsDesc()
SLE_VAR(GoodsEntry, last_age, SLE_UINT8),
SLEG_CONDVAR( _cargo_feeder_share, SLE_FILE_U32 | SLE_VAR_I64, 14, 64),
SLEG_CONDVAR( _cargo_feeder_share, SLE_INT64, 65, 67),
+ SLE_CONDVAR(GoodsEntry, amount_fract, SLE_UINT8, 149, SL_MAX_VERSION),
SLE_CONDLST(GoodsEntry, cargo.packets, REF_CARGO_PACKET, 68, SL_MAX_VERSION),
SLE_END()
diff --git a/src/station_base.h b/src/station_base.h
index ca0e598f9..d595379e9 100644
--- a/src/station_base.h
+++ b/src/station_base.h
@@ -42,6 +42,7 @@ struct GoodsEntry {
byte rating;
byte last_speed;
byte last_age;
+ byte amount_fract; ///< Fractional part of the amount in the cargo list
StationCargoList cargo; ///< The cargo packets of cargo waiting in this station
};
diff --git a/src/station_cmd.cpp b/src/station_cmd.cpp
index 74b4f484b..b06e2a874 100644
--- a/src/station_cmd.cpp
+++ b/src/station_cmd.cpp
@@ -3190,7 +3190,13 @@ void ModifyStationRatingAround(TileIndex tile, Owner owner, int amount, uint rad
static void UpdateStationWaiting(Station *st, CargoID type, uint amount, SourceType source_type, SourceID source_id)
{
- st->goods[type].cargo.Append(new CargoPacket(st->index, st->xy, amount, source_type, source_id));
+ amount += st->goods[type].amount_fract;
+ st->goods[type].amount_fract = GB(amount, 0, 8);
+
+ /* No new "real" cargo item yet. */
+ if (amount <= UINT8_MAX) return;
+
+ st->goods[type].cargo.Append(new CargoPacket(st->index, st->xy, amount >> 8, source_type, source_id));
SetBit(st->goods[type].acceptance_pickup, GoodsEntry::PICKUP);
TriggerStationAnimation(st, st->xy, SAT_NEW_CARGO, type);
@@ -3328,11 +3334,14 @@ uint MoveGoodsToStation(CargoID type, uint amount, SourceType source_type, Sourc
/* no stations around at all? */
if (st1 == NULL) return 0;
+ /* From now we'll calculate with fractal cargo amounts.
+ * First determine how much cargo we really have. */
+ amount *= best_rating1 + 1;
+
if (st2 == NULL) {
/* only one station around */
- uint moved = amount * best_rating1 / 256 + 1;
- UpdateStationWaiting(st1, type, moved, source_type, source_id);
- return moved;
+ UpdateStationWaiting(st1, type, amount, source_type, source_id);
+ return amount >> 8;
}
/* several stations around, the best two (highest rating) are in st1 and st2 */
@@ -3340,26 +3349,19 @@ uint MoveGoodsToStation(CargoID type, uint amount, SourceType source_type, Sourc
assert(st2 != NULL);
assert(best_rating1 != 0 || best_rating2 != 0);
- /* the 2nd highest one gets a penalty */
- best_rating2 >>= 1;
-
- /* amount given to station 1 */
- uint t = (best_rating1 * (amount + 1)) / (best_rating1 + best_rating2);
+ /* Then determine the amount the worst station gets. We do it this way as the
+ * best should get a bonus, which in this case is the rounding difference from
+ * this calculation. In reality that will mean the bonus will be pretty low.
+ * Nevertheless, the best station should always get the most cargo regardless
+ * of rounding issues. */
+ uint worst_cargo = amount * best_rating2 / (best_rating1 + best_rating2);
+ assert(worst_cargo < (amount - worst_cargo));
- uint moved = 0;
- if (t != 0) {
- moved = t * best_rating1 / 256 + 1;
- amount -= t;
- UpdateStationWaiting(st1, type, moved, source_type, source_id);
- }
-
- if (amount != 0) {
- amount = amount * best_rating2 / 256 + 1;
- moved += amount;
- UpdateStationWaiting(st2, type, amount, source_type, source_id);
- }
+ /* And then send the cargo to the stations! */
+ UpdateStationWaiting(st1, type, amount - worst_cargo, source_type, source_id);
+ UpdateStationWaiting(st2, type, worst_cargo, source_type, source_id);
- return moved;
+ return amount >> 8;
}
void BuildOilRig(TileIndex tile)