Deliver fetch quest items via truck - #88308
Open
RenechCDDA wants to merge 1 commit into
Open
Conversation
RenechCDDA
force-pushed
the
deliver_items_via_truck
branch
from
August 12, 2026 21:49
e5d896c to
5d72ff1
Compare
Including... -Hashing function for item_location (std::unordered_set support) -A map function to JUST GIVE ME ALL THE DAMN ITEMS AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA -We draw a little happy tree in f_consume_item_sum() as we refactor :)
RenechCDDA
force-pushed
the
deliver_items_via_truck
branch
from
August 12, 2026 22:24
5d72ff1 to
babb0dd
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Features "Deliver fetch quest items via truck"
Purpose of change
Piling up solar panels on the ground is stupid
Describe the solution
Make it less stupid. They can come from your truck.
Includes:
-Hashing function for item_location (std::unordered_set support)
-A map function to "just give me the damn items" Cause items can be:
On a person
In a vehicle
On a map
Inside another item on one of the above
And all of that is terrible.
-Minor refactoring in f_consume_item_sum() to use the new function (and also because it was very difficult to follow)
Describe alternatives you've considered
Delete all pointless fetch quests
Testing
Balthazar ate 12 solar panels from me in each of the following combinations:
12 in truck
6 in truck, 6 in inventory
12 on ground
4 in truck, 4 in in inventory, 4 on ground
Additional context