Robert Hensing 61a4d3d45c getSnippetUpTo: Return optional ...

This makes it possible to certain discern failures from empty
snippets, which I think is an ok review comment.

Maybe it should do so for swapped column indexes too, but I'm not
sure.

I don't think it matters in the grand scheme. We don't even have
a real use case for `nullopt` now anyway.

Since we don't have a use case, I'm not applying this logic to
higher level functions yet.

2024-07-15 20:10:45 +02:00

3.4 KiB

Raw Blame History

View Raw