From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Berry, Charles" Subject: org-babel-insert-result doesn't comma escape properly Date: Wed, 21 Aug 2019 23:28:40 +0000 Message-ID: <94FB08FE-1466-4B0A-B544-06A14E70114C@ucsd.edu> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:470:142:3::10]:49211) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i0a2Z-0007LN-0s for emacs-orgmode@gnu.org; Wed, 21 Aug 2019 19:29:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1i0a2W-0002H6-Lf for emacs-orgmode@gnu.org; Wed, 21 Aug 2019 19:29:17 -0400 Received: from mx0a-00395d01.pphosted.com ([148.163.133.170]:19292) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1i0a2W-0002GS-4A for emacs-orgmode@gnu.org; Wed, 21 Aug 2019 19:29:16 -0400 Received: from pps.filterd (m0173167.ppops.net [127.0.0.1]) by mx0a-00395d01.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id x7LNE97k027397 for ; Wed, 21 Aug 2019 16:29:12 -0700 Received: from secdlpezp02.ucsd.edu ([40.112.251.180]) by mx0a-00395d01.pphosted.com with ESMTP id 2uhashgebx-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Wed, 21 Aug 2019 16:29:12 -0700 Content-Language: en-US Content-ID: <41556A2F15F91742BC4DEC07DB731574@AD.UCSD.EDU> List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: "Emacs-orgmode" To: Org Mode Here is an ECM: #+begin_src emacs-lisp :wrap example "line 1 ,* headline 2 ,* headline 3 ,* headline 4 ,* headline 5 " #+end_src With today's master, the last `headline' is not escaped in the example bloc= k this produces when executed. It seems to me that dropping the let binding for `before-finish' and placin= g (unless no-escape (org-escape-code-in-region beg end)) before the insertions of 'start' and 'finish' should handle this properly. = And trying that on the above ECM gives the right result. That let binding was introduced in=20 commit 24a76fbe572923c55774bc9f8ecc8e6d1c7ff16d Author: Nicolas Goaziou Date: Sun Aug 13 16:20:20 2017 +0200 I do not see why it was needed. Best, Chuck=