Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
efead0a
Parts of implementation using ada-url instead of uriparser
azrogers Jan 15, 2025
b397b06
Merge branch 'main' of github.com:CesiumGS/cesium-native into ada-url
azrogers Jan 16, 2025
9617622
Part of a Uri refactor
azrogers Jan 17, 2025
ad542c1
It's working!?
azrogers Jan 17, 2025
6fe2fdf
Document methods
azrogers Jan 17, 2025
f57f7d9
Remove cruft
azrogers Jan 17, 2025
45d9d7f
Merge from main
azrogers Jan 17, 2025
6115d11
Format
azrogers Jan 17, 2025
eadf419
Fix clang-tidy issues
azrogers Jan 17, 2025
887f981
Fix Doxygen issues
azrogers Jan 17, 2025
68aebad
Remove find_package(uriparser)
kring Jan 19, 2025
78f0f40
Re-add uriparser for now, and fix macOS compile error.
kring Jan 19, 2025
e777426
Fix CI issues
azrogers Jan 21, 2025
38c1e31
Format
azrogers Jan 21, 2025
ea46377
Rewrite substituteTemplateParameters to work with encoded brackets
azrogers Jan 23, 2025
605ea5a
Format
azrogers Jan 23, 2025
40d4ba9
Merge branch 'main' of github.com:CesiumGS/cesium-native into ada-url
azrogers Jan 23, 2025
84cc7c7
Merge branch 'main' of github.com:CesiumGS/cesium-native into ada-url
azrogers Jan 23, 2025
8c84337
A few more tests for edge cases, regex implementation
azrogers Jan 24, 2025
4b3e558
Merge branch 'main' of github.com:CesiumGS/cesium-native into ada-url
azrogers Jan 24, 2025
369d3a8
Merge remote-tracking branch 'origin/main' into ada-url
kring Jan 28, 2025
17eefa1
Substitute placeholders before manipulating layer.json URLs.
kring Jan 28, 2025
158de7e
Merge branch 'main' of github.com:CesiumGS/cesium-native into ada-url
azrogers Jan 28, 2025
9789911
Address most of review
azrogers Jan 28, 2025
9e032f8
Merge pull request #1084 from CesiumGS/ada-url-substitute-first
azrogers Jan 28, 2025
cb18f76
Return to previous substitution logic
azrogers Jan 28, 2025
7006b81
Remove uriparser, format, etc
azrogers Jan 28, 2025
3d5d39d
Update dependency graphs
azrogers Jan 28, 2025
c3a18c6
Format
azrogers Jan 28, 2025
1c4e4c2
Merge from main
azrogers Jan 29, 2025
db5868a
Address review comments
azrogers Jan 29, 2025
46ad1ac
Separate query handling from Uri
azrogers Jan 29, 2025
cea035d
Fix clang-tidy header warning
azrogers Jan 30, 2025
b0a16d2
Fix docs
azrogers Jan 30, 2025
f34c8fe
Format
azrogers Jan 30, 2025
eb016f0
Rename UriQueryParams -> UriQuery
azrogers Jan 31, 2025
9263109
Merge remote-tracking branch 'origin/main' into ada-url
kring Feb 1, 2025
8d319e4
Correct comment that is no longer true.
kring Feb 1, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix Doxygen issues
  • Loading branch information
azrogers committed Jan 17, 2025
commit 887f9816a72b7f98ead8bd655a760b94e6d0df2c
6 changes: 3 additions & 3 deletions CesiumUtility/include/CesiumUtility/Uri.h
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ class Uri final {
* @brief Sets the path portion of a URI to a new value. The other portions of
* the URI are left unmodified, including any query parameters.
*
* @param newPath The new path portion of the URI.
* @param path The new path portion of the URI.
*/
void setPath(const std::string_view& path);

Expand Down Expand Up @@ -157,7 +157,7 @@ class Uri final {
* if not found.
*
* @deprecated Create a \ref Uri instance and use \ref
* Uri::getQueryValue(const std::string& key) instead.
* Uri::getQueryValue(const std::string&) instead.
*/
static std::string
getQueryValue(const std::string& uri, const std::string& key);
Expand Down Expand Up @@ -310,7 +310,7 @@ class Uri final {
* parsed, it is returned unmodified.
*
* @deprecated Create a \ref Uri instance and use \ref Uri::setPath(const
* std::string_view& path) instead.
* std::string_view&) instead.
*/
static std::string
setPath(const std::string& uri, const std::string& newPath);
Expand Down
Loading