> On 4 Aug 2025, at 17:18, Fujii Masao <masao.fujii@gmail.com> wrote:
I missed this thread while being on vacation, thanks for finding and fixing
this!
> This also got me thinking, if we simply define keywords as strings of
> non-whitespace characters, maybe we don't need to change the term "keyword"
> to "token" at all. I've updated the patch with that in mind. Thoughts?
Agreed, this should work fine, and it aligns the code somwhat with read_pattern
which is a good thing.
+ * in line buffer. Returns NULL when the buffer is empty or no keyword exists.
Since "is empty" could be interpreted as being a null pointer, maybe we should
add a if (!*line) check (or an Assert) before we dereference the passed in
buffer?
--
Daniel Gustafsson
