Replace uriparser
usage with CesiumUtility::Uri
#1589
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.
With
uriparser
getting replaced withada
in CesiumGS/cesium-native#1072, we need to remove the usage ofuriparser
in Cesium for Unreal so we can remove it from the dependencies entirely. This change utilizes theCesiumUtility::Uri::uriPathToNativePath
andCesiumUtility::Uri::getPath
methods to resolve file URIs. As part of it, I needed to wrap includes ofCesiumUtility/Uri.h
inTHIRD_PARTY_INCLUDES_START
and_END
macros asada
references multiple macros in#if
directives, such asADA_USING_LIBRARY
, that aren't necessarily defined. This causes errors on MSVC. TheTHIRD_PARTY_INCLUDES_
macros ignore errors in the included header files.