Refactor classic ai into tbc#8
Open
hlarsen wants to merge 14 commits intoblueboy:masterfrom
hlarsen:refactor-classic-ai-into-tbc
Open
Refactor classic ai into tbc#8hlarsen wants to merge 14 commits intoblueboy:masterfrom hlarsen:refactor-classic-ai-into-tbc
hlarsen wants to merge 14 commits intoblueboy:masterfrom
hlarsen:refactor-classic-ai-into-tbc
Conversation
+Eliminates a mismatch between quest progress status and player fields for items +Add comments with corresponding UI events
Default Mac OpenSSL directory to home-brew installation path if nothing else is specified. (+9 squashed commits) Squashed commits: [6d2ce1c] Update CMakeLists.txt [263d9ba] Update loadlib.h Copied from CMaNGOS defs [eb0b9f9] Update loadlib.h [1f510c7] Update loadlib.h [2461169] Update .travis.yml [ecc939d] Update .travis.yml [b88d7b1] Update .travis.yml [3cd9472] Update .travis.yml [3f0b716] Update .travis.yml [ff219c0] Update .travis.yml
Fix Mac assert macro weirdness Use APPLE instead of checking if systemname matches Darwin, cleaner
need to check and update the queries and then we'll be ready to test
… and fixes for updated playerbot code
bots were continually trying to loot corpses which only had items tagged for other players. now we simply check that the corpse has items we can loot and if not we're done.
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
6 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.
this pr finishes bringing the repo up to date with cmangos/mangos-tbc@master, as well as ports over the various bot ai updates. please note that the code had diverged so much I didn't bother trying to merge the tbc spells into the updated ai, however I did leave the tbc spell defs in the class ai files in case someone wants to take a crack at it. this brings the tbc and classic bots a lot closer together, however a few big deferences still exist as tbc bots have autofollow and autoequip functionality it appears the classic bots to not, or was implemented differently.
I did some light testing with a warrior and priest bot and they seemed to correctly obey combat orders, but I didn't get that in depth.