Improve scalar_style_choose() to account for flow/block mode:#606
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #606 +/- ##
==========================================
+ Coverage 98.84% 99.08% +0.23%
==========================================
Files 43 43
Lines 13789 13869 +80
==========================================
+ Hits 13630 13742 +112
+ Misses 159 127 -32 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
biojppm
force-pushed
the
scalar_style_choose
branch
2 times, most recently
from
5dce9ae to
a97edb8
Compare
this enables stable-style roundtrips of block-mode plain scalars with characters invalid in flow mode:
```yaml
# these scalars were previously emitted as single-quoted style
- doe: a deer, a female deer
- (10,11)
- also scalars containing [] and {}
```
- `scalar_style_choose()` was split into `scalar_style_choose_{flow,block}()`
- `scalar_style_query_plain()` was split into `scalar_style_query_plain_{flow,block}()`
- `NodeType::type_str(NodeType,flags)`: remove zero-termination, use common approach of returning required size
- Add `NodeType::type_str_sub()`
biojppm
force-pushed
the
scalar_style_choose
branch
from
a97edb8 to
580ebb8
Compare
Hancock33
added a commit
to Hancock33/batocera.piboy
that referenced
this pull request
May 23, 2026
----------------------------------------------------------------------------------- eden.mk 9b18d0b1113fb2b73acc539d2e4dfae85e511cd4 # Version: Commits on May 22, 2026 ----------------------------------------------------------------------------------- [tools] refactor, use #!/bin/sh, update license files (#3998) ----------------------------------------------------------------------------------- ymir.mk 9f3425a1cd8f70e731179b0ca3fca04b0489d28c # Version: Commits on May 22, 2026 ----------------------------------------------------------------------------------- chore(app): Removed graphics backend option as it served no purpose at this time Ymir will rely on the default backend used by the SDL3 Renderer API, which should be compatible with most systems. GPU rendering will include a rewrite of the app's graphics subsystem, moving away from SDL Renderer to our own implementation., ------------------------------------------------------- rapidyaml.mk v0.13.0 # Version: Commits on May 22, 2026 ------------------------------------------------------- - [PR#606](biojppm/rapidyaml#606): scalar style utilities now are specific to flow/block mode. This enables stable-style roundtrips of block-mode plain scalars with characters invalid in flow mode: ```yaml # these scalars were previously emitted as single-quoted style # because they were conservatively assumed to be in flow mode. - doe: a deer, a female deer - (10,11) - also scalars containing [] and {} ``` - `scalar_style_choose()` was split into `scalar_style_choose_{flow,block}()` - `scalar_style_query_plain()` was split into `scalar_style_query_plain_{flow,block}()` - `NodeType::type_str(NodeType,flags)`: remove zero-termination, use common approach of returning required size - Add `NodeType::type_str_sub()` - [PR#605](biojppm/rapidyaml#605): fix parse errors: - `:` and `#` on continuation lines of multiline plain scalars: ```yaml [plain :scalar] # leading colon belongs to the scalar ``` - Tabs inside quoted scalars opening a map were mistook for indentation tabs: ```yaml foo: '\\ta': b ``` - [PR#604](biojppm/rapidyaml#604): add `string_view` and `span` to the `c4/yml/std/std.hpp` interop umbrella header. - Fix [600](biojppm/rapidyaml#600): shared symbols not exported in clang on Windows ([PR#601](biojppm/rapidyaml#601)). - Fix [256](biojppm/rapidyaml#256): installation directory on Linux 64bit ([PR#599](biojppm/rapidyaml#599)). See also original fix at [cmake#16](biojppm/cmake#16). Big thanks to @GabrielBarrantes and @musicinmybrain, not just for their fixes but also for all their downstream work! - [PR#591](biojppm/rapidyaml#591): Add missing includes to avoid compilation warning. Thanks @GabrielBarrantes! - Update c4core to [0.3.0](https://github.com/biojppm/c4core/releases/tag/v0.3.0). ### Thanks - @GabrielBarrantes - @musicinmybrain , ------------------------------------------------------------------------------------------------- libretro-geargrafx.mk 15e31680428046281b4457d126f503d8ab8e2f8f # Version: Commits on May 22, 2026 ------------------------------------------------------------------------------------------------- Fix scsi audio command completion. Fix #125, ----------------------------------------------------------------------------------------------------- libretro-mame2003-plus.mk 45f7a7f89f60aa1da6e6a136bbc1d1adeea8af91 # Version: Commits on May 22, 2026 ----------------------------------------------------------------------------------------------------- Rom ignore (#2022) * add rom ignore * fix gng romloads, ---------------------------------------------------------------------------------------------- libretro-stella.mk 7cb8757b44e563e67cdd4a59be19aab7e2b139ee # Version: Commits on May 22, 2026 ---------------------------------------------------------------------------------------------- Cleaned up EventMappingWidget, removing some FIXME's.,
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
scalar_style_choose()was split intoscalar_style_choose_{flow,block}()scalar_style_query_plain()was split intoscalar_style_query_plain_{flow,block}()NodeType::type_str(NodeType,flags): remove zero-termination, use common approach of returning required sizeNodeType::type_str_sub()