Update return type of LogCategory::get_category_names() from std::vector<const char*> to std::vector<std::string_view>

[elle-j]

What, How & Why?

Updates the return type of LogCategory::get_category_names() from std::vector<const char*> to std::vector<std::string_view> in order to alleviate marshaling of strings in DotNet.

This also aligns better with LogCategory::get_category(std::string_view name) (already taking a std::string_view).

We don't believe this will be a breaking change.

SDK reviewers, please confirm if this is non-breaking/breaking for you.

☑️ ToDos

• 📝 Changelog update
• 🚦 Tests (or not relevant)
• C-API, if public C++ API changed
  • Remains unchanged.
• bindgen/spec.yml, if public C++ API changed
  • This API was never exposed.

[coveralls-official]

Pull Request Test Coverage Report for Build linnea.jansson_7

Details

• 7 of 7 (100.0%) changed or added relevant lines in 2 files are covered.
• 98 unchanged lines in 19 files lost coverage.
• Overall coverage decreased (-0.02%) to 90.981%

---

Files with Coverage Reduction	New Missed Lines	%
src/realm/array_mixed.cpp	1	94.02%
src/realm/index_string.hpp	1	93.48%
src/realm/util/serializer.cpp	1	90.43%
test/fuzz_tester.hpp	1	57.73%
test/test_dictionary.cpp	1	99.83%
test/test_table.cpp	1	99.51%
src/realm/list.cpp	2	87.37%
src/realm/object-store/shared_realm.cpp	2	91.89%
src/realm/sync/network/http.hpp	2	82.27%
test/test_lang_bind_helper.cpp	2	93.2%

Totals
Change from base Build 2483:	-0.02%
Covered Lines:	215223
Relevant Lines:	236558

---

💛 - Coveralls

[clementetb]

I didn't find any incompatibility with the Kotlin SDK.

[dianaafanador3]

It is working as expected in Swift