John Bartholomew
John Bartholomew
#1142 is one possible approach to prevent this particular stack overflow.
A few open bugs should be fixed just by upgrading Rapid YAML: - #942 (which has a contributed but not-yet-merged test case #943) - #1014 - #1050 YAML-parsing-related bug #1109...
Currently breaks tests because some of the test data files have a `.cpp` extension which is caught by the auto format command. Anyway, I'm going to postpone this PR for...
Uh... this is going to bitrot fast, because big reformat PRs always do. I'll just close it. Perhaps we can reformat incrementally instead.
@CertainLach Any response to my [previous comment](https://github.com/google/jsonnet/pull/1119#pullrequestreview-1903683025)? How would you feel about implementing this as a builtin instead?
The fuzz generated input is basically a large number of opening parentheses. It has a few other characters mixed in, but that's not actually very interesting, the stack overflow can...
Looks like an oversight - we should add the dependency conditionally. Thanks for the bug report!
Actually looking closer it looks like it's just an entirely incorrect dependency and should be removed. I will do some more digging to try to work out if I've misunderstood...
It seems like this was almost ready to go back in June, with only a request for documentation updates as a possible blocker. @Duologic would you like to contribute some...
Hmm... it's a good point, the tutorial says "Array slices like `arr[10:20:2]` are allowed, like in Python", so perhaps people have always assumed negative slices worked :-) IMHO, yes, it...