Basil L. Contovounesios <contovob@tcd.ie> writes:
> Only a minor question from me, see below.
> [...]
> Why is buffer-string called twice in this and the following two tests?

Thank you for your input.  These calls were left there by mistake, fixed in
the
attached patch.

> You said you were disinclined to adapt delim-col.el itself, but how
> about bundling the following minor cleanups as well?

Looks good to me.  I've included your cleanups in the attached patch.

Noam Postavsky <npostavs@gmail.com> writes:
> I think you may as well go with defun here, delimit-columns-str isn't
> being used in some performance-critical spot, as far as I can tell.

You are probably correct.  However, I decided to leave Basil's changes as
is,
since I don't know what would conventionally be considered cleaner/better in
this case.

Stefan Kangas