|
I wonder why they did it in that convoluted way. It could have been much easier to introduce GetBagSize(BAG_BANK) and GetUsableBagSize(BAG_BANK), where the former one indicates the slot range and the latter one the number of slots that can be filled at the same time, like
Without ESO+:
GetBagSize(BAG_BANK) = 500
GetUsableBagSize(BAG_BANK) = 250
Then, with ESO+:
GetBagSize(BAG_BANK) = 500
GetUsableBagSize(BAG_BANK) = 500
Then, deny transfers to the bank if the fill state reached or exceeded GetUsableBagSize(BAG_BANK). Mission accomplished.
Or, simply leave it with GetBagSize(BAG_BANK), have its return value doubled when ESO+ is active, but lift the constraint that the slot table indices are numbered 1..(size).
|