Updating forked master #1
Open
Conversation
This file contains 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
The binary has to be run from the bin directory, otherwise the DroidSans font can't be found and shows the error "Could not init GUI renderer.": https://github.com/recastnavigation/recastnavigation/blob/master/RecastDemo/Source/main.cpp#L129
* Don't install test * Install to the correct directory (e.g. lib64) * Install includes to /usr/include/recastnavigation
* Add -d suffix for debug libraries * Export PDB files for each Debug library
* minor redundant: dir * bugfix: ChunkyTriMesh nodes array: out of bounds
* Validate constraint: #polygons per tile When adding a tile to a NavMesh, ensure that the number of polygons in that tile fit in the poly ID address space. * Improve dtNavMeshParams field documentation
* Add dtNavMeshQuery.findNearestPoly() overload with parameters: distance, isOverPoly * dtNavMeshQuery::findNearestPoly(): improve documentation, avoid code duplication * Make code C++98-compatible * Remove distance parameter from dtNavMeshQuery::queryPolygons()
* Use clear() instead of resize(0) for temp vectors Nicer style * rcVectorBase::resize: Avoid realloc on grow When capacity is sufficient, do not realloc the whole buffer * rcVector: Grow exponentially on resize
* Add version and pkgconfig * simply versioning Co-authored-by: Bret Curtis <[email protected]>
* use defaults where possible, simplify .travis.yml and fix macos builds while we are at it * de-duplicate code * specifically ask for xcode12.2
* Set hints to always use OpenGL render driver (instead of Metal driver) * Compile error fix in Xcode 13
Thanks to @pineappleKID for pointing this out! #535
The return value should be the perp dot product rather than the dot product.
fix typo though
…y::findRandomPoint (#560)
The premake script generates some deprecation warnings when generating with premake5: - "configuration" has been deprecated in favor of "filter" - "solution" was renamed to "workspace"
…r does it internally. Having two GL contexts causes rendering problems on Ubuntu. Fixes #521
…ions of GCC Based on SgtVincent's suggestion here: #497 (comment) Fixes #497
https://github.com/catchorg/Catch2 Updated tests with new API changes as well.
Updated appveyor version matrix. Removed vs2013
Catch seems to only work with VS 2017+
…for C (#479) * Added a selective filter for *.c files compiled on linux using make Co-authored-by: Graham Pentheny <[email protected]>
Also remove hard-coded filter on gmake premake target since it's deprecated in favor of gmake2. It also doesn't matter; what matters is that we filter on gcc, since the warnings we've disabled don't exist in clang and will throw errors.
* Fixes for many low-severity compiler warnings Mostly a lot of pedantic things like end-of-file newlines and commas on the last enum value. However some fixes like the weak v-tables warnings fixes might help with code gen. It's unclear if the linker is able to elide multiple copies of the same type's v-table in different translation units, (sometimes it can) but these fixes ensure we don't have to rely on it.
This makes the default xcode project generated by premake work out of the box, and is generally easier for mac users. Especially those who don't have permissions to modify /Library/Frameworks/ - Look for SDL in RecastDemo/Bin by default rather than /Library/Frameworks/ - Updated README build instructions - Ignore SDL2.framework in RecastDemo/Bin
The goal here is to replace the current (slightly broken) Travis + Appveyor setup with something that covers more cases and is a bit better supported and easier to maintain. This hopefully helps us catch cross-platform issues quicker. For example, the linux and clang compilation issues that have existed for a while could have been caught if we were building those targets in CI.
This adds a build script that builds the following configurations for every repo commit and PR:
macOS, premake, Debug
macOS, premake, Release
linux, premake, gcc, Debug
linux, premake, gcc, Release
linux, premake, clang, Debug
linux, premake, clang, Release
linux, cmake, Debug
linux, cmake, Release
windows, premake, vs2019, Debug
windows, premake, vs2019, Release
windows, premake, vs2022, Debug
windows, premake, vs2022, Release
windows, cmake, vs2019, Debug
windows, cmake, vs2019, Release
windows, cmake, vs2022, Debug
windows, cmake, vs2022, Release
It also builds and runs the catch tests executable in the following configurations. A failed test will fail the build.
macOS, premake, Debug
linux, premake, clang, Debug
windows, premake, vs2022, Debug
It doesn't currently build cmake on macOS because there is a one blocker there (#577). We could also add additional builds like cmake with both clang and gcc on linux, but for now this is much better coverage than we have currently.
- Upgraded doxyfile to latest format - Fixed a bunch of warnings and errors in docstrings that doxygen was complaining about - Added the doxygen-awesome theme to modernize the output - Removed a duplicate screenshot we'd had in the docs folder - Moved the RecastDemo screenshot into the Docs/images folder - Changed the doxygen main page to be the README.md since the dedicated main page file we had was nearly identical but out of date - Added CONTRIBUTING.md so it's part of the generated doxygen output - Removed duplicate license file to avoid confusion - Combined the FAQ with the Recast_api.txt, since the FAQ was essentially just better docs for the members of `rcConfig`
* Fix incorrect rasterization at tile borders Clip the polygon at the tile border and disregard coordinates outside of the tile before clamping. * Added unit test with minimal repro case for the rasterization issue fixed in PR #476 Co-authored-by: Graham Pentheny <[email protected]>
Specifically, when rasterizing a triangle into a heightfield with width or height of 0. This can happen when the input geometry bounding box extents are less than half the cell size in either x or z. This is a degenerate case, but can crop up in real life scenarios (see godotengine/godot#65764) so we should at least ensure it doesn't crash here.
* Ignore cmake intermediate build dir * Fix C++ language version in cmake build scripts * Set a default SDL2_ROOT_DIR value for macOS in cmake scripts * Fixed macOS SDL framework linking in cmake scripts * Added macos-cmake target for CI builds in github workflows * Use C++17 for building Tests instead of 20, since we build them with VS2019 in some jobs
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.
This PR is to update this fork repo
This change is