else: newlist is uninitialized !
RlistFromSplitRegex() returns NULL if given a NULL string (but asserts against that) or if given an empty string.
So I think `newlist = file_buffer ? RlistFromSplitRegex(...) : NULL;` would be sensible.