Update Develop Branch - #1
Merged
Bobisback merged 261 commits intoJun 13, 2024
Merged
Conversation
… issues The exe still need extra padding units for savegame compatibility. Now the first entry is used instead.
somebody loved playing for Russia, but did not like the difficulties that Russian colonists faced.
…gration threshold is overrun.
3 more lines are added to assert message (file, line, function of caller to the failing function). This requires an added argument in function calls to support this, so it's not added everywhere. Added this to functions known to contain assert failures where the cause has yet to be determined.
…mmunity member. Did not change the button in the atlas though, as it is corrupted anyways somehow.
… or Valuable Wood to be generated (due to having a River) on plots that usually do not have it at all.
Added additional safeguards and assert checks to catch issues like this in the future!
+ fixed missing text on the "promotion" button. again duplication and use of TXT_KEY instead of text.
…nt isn't in the event trigger queue CvPlayer::applyEvent has been rewritten to use an EventTrigger reference instead of an ID. This avoids all NULL reference issues and can make it work on event triggers of any source. Added overloaded CvPlayer::applyEvent to safely convert to EventTrigger reference. Got rid of all EventTrigger->ID->EventTrigger when calling CvPlayer::applyEvent.
… negative. The Tailwind event applies negative movement point so we may as well disable the check since negative movements points are by now a "sanctioned feature"
typo correction + translation correction
+ correction
…ontain Latin characters
…s with turn damage. The AI now relies on pathCost to make such moves unattractive due to high cost. Removed commented out code
…system aka NMS): Corrected the calculation of path cost for the NMS case (iWorstMovesLeft was not set at all!) The MOVE_HAS_STEPPED pf flag is now set when a unit has moved (was left out by oversight) Enabled the path cost modification for exploring units The path cost for paths that would cause damage is now taken into account for the entire path and not just for the endplot.
…* iMoveCost when using NMS
… more than 3 starting units Note: Currently only affected the Spanish
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
11 participants
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.
No description provided.