Subject: Re: bug#55315: [elpa/csv-mode] [PATCH] CSV separator guessing

From: Mattias Engdegård <[email-masked]>
Date: Mon, 9 May 2022 11:37:41 +0200

8 maj 2022 kl. 21.31 skrev Simen Heggestøyl <[email-masked]>:

> Am I right in thinking that the use of
> `regexp-quote' in the `csv-set-separator' case gives the right result?

Yes, I think so. `csv-set-separator` should probably escape the character in `csv--skip-chars`, however:

  (setq-local csv--skip-chars (format "^\n%c" sep))

should be

  (setq-local csv--skip-chars (format "^\n\\%c" sep))

I'm not sure if a separator can be chosen that needs escaping here but better be safe; who knows how the code will be used.