snakeyaml-engine-kmp
snakeyaml-engine-kmp copied to clipboard
krzema12
A Kotlin Multiplatform port of snakeyaml-engine
Fix minor warnings, including but not limited to: - Unnecessary boxing - 'null' is returned by the method declared as `@NotNull` - Not annotated method overrides method annotated with `@NotNull`
This issue lists Renovate updates and detected dependencies. Read the [Dependency Dashboard](https://docs.renovatebot.com/key-concepts/dashboard/) docs to learn more. --- > [!WARNING] > Renovate failed to look up the following dependencies: `Failed to...
![renovate[bot] avatar](https://avatars.githubusercontent.com/in/2740?v=4 "renovate[bot] avatar"){kind=avatar}
[](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [io.github.typesafegithub:github-workflows-kt](https://togithub.com/typesafegithub/github-workflows-kt) | `1.15.0` -> `2.0.0` | [](https://docs.renovatebot.com/merge-confidence/)...
- remove KT-51379 workaround - add `.kotlin/` to `.gitignore` - update `yarn.lock`
cherry picked from upstream commit [ef7ebe9c06e963e13f4ab465f300a0dd6f0940c9](https://bitbucket.org/snakeyaml/snakeyaml-engine/commits/ef7ebe9c06e963e13f4ab465f300a0dd6f0940c9) SnakeKMP isn't affected by [the issue](https://bitbucket.org/snakeyaml/snakeyaml-engine/issues/51/find-a-way-to-dump-as-json), but more tests don't hurt. I'm planning to refactor StreamReader, so this test will be useful.
There's no need to skip BOMs in ScannerImpl. ScannerImpl gets the characters from StreamReader, which skips BOMs. Skipping them again here is redundant.
There are currently no direct tests for StreamReader. If a bug is introduced into StreamReader, it can be hard to track down the root cause, because unrelated tests will fail....
- Change codepoint buffer from mutable `IntArray` to read-only `List` - Deprecate alternate constructors. They were only used in tests, and the 'real' constructor wasn't tested. - update kdoc -...
- Tidy up kdoc. - Deprecate and rename functions to use overloaded functions instead. - Add default arg for LoadSettings (most usages just use the default, it saves a lot...
Avoid eagerly fetching data. Use ArrayDeque as a buffer instead of custom array with manual index management.