From: charles@aurox.ch (Charles A. Roelli)
To: John Palmieri <jhpalmieri64@gmail.com>, 21778-done@debbugs.gnu.org
Subject: bug#21778: 24.5; python mode: fill-paragraph with raw docstring ignores python-fill-docstring-style
Date: Wed, 16 Aug 2017 17:05:12 +0200 [thread overview]
Message-ID: <m27ey3r1sn.fsf@aurox.ch> (raw)
In-Reply-To: <CAHNEuZzTfFHPcuyt8fnzUKmw8f9c=AQiGday7pBP81DmJiwwpw@mail.gmail.com> (message from John Palmieri on Wed, 16 Aug 2017 07:56:46 -0700)
Thanks, closing the bug.
> From: John Palmieri <jhpalmieri64@gmail.com>
> Date: Wed, 16 Aug 2017 07:56:46 -0700
>
>
> Yes, thank you very much, it looks like the bug has been fixed in Emacs 25.
>
> On Wed, Aug 16, 2017 at 2:09 AM, Charles A. Roelli <charles@aurox.ch> wrote:
>
> Looks like this was fixed with the following commit:
>
> commit deea36f0ece7b1b14afe2a833b6a0f66d59f4459
> Author: Fabián Ezequiel Gallina <fgallina@gnu.org>
> Date: Sun Apr 5 23:58:13 2015 -0300
>
> python.el: Enhance docstring detection following PEP-257.
>
> * lisp/progmodes/python.el (python-docstring-at-p): Remove function.
> (python-info-assignment-statement-p): New function.
> (python-info-assignment-continuation-line-p): Use it.
> (python-info-docstring-p): New function.
> (python-font-lock-syntactic-face-function)
> (python-fill-string): Use it.
>
> * test/automated/python-tests.el (python-info-assignment-statement-p-1)
> (python-info-assignment-statement-p-2)
> (python-info-assignment-statement-p-3, python-info-docstring-p-1)
> (python-info-docstring-p-2, python-info-docstring-p-3)
> (python-info-docstring-p-4, python-info-docstring-p-5)
> (python-info-docstring-p-6): New tests.
>
> which is present in Emacs 25. Does the fix work for you?
>
> > Resent-From: John Palmieri <jhpalmieri64@gmail.com>
> > Original-Sender: "Debbugs-submit" <debbugs-submit-bounces@debbugs.gnu.org>
> > Resent-CC: bug-gnu-emacs@gnu.org
> > Resent-Sender: help-debbugs@gnu.org
> > X-Spam-Status: No, score=0.3 required=5.0 tests=BAYES_40,
> > FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,HTML_MESSAGE,T_DKIM_INVALID
> > autolearn=disabled version=3.3.2
> > Date: Wed, 28 Oct 2015 12:16:20 -0700
> > From: John Palmieri <jhpalmieri64@gmail.com>
> > Content-Type: multipart/alternative; boundary=001a113fe84e88f40805232f076f
> >
> >
> > To recreate the bug, in Python mode, type
> >
> > def f():
> > r"""
> > This is the docstring for my function.
> >
> > It's a raw docstring because I want to type \t here, and maybe \n,
> > for example in LaTeX code like \tau or \nu.
> >
> > More docstring here.
> > """
> > pass
> >
> > Now with the cursor in the first sentence of the docstring, running
> > fill-paragraph ignores the setting of python-fill-docstring-style, I think
> > because Emacs doesn't detect that r""" is the start of the docstring.
> > Similarly, running fill-paragraph in the last sentence behaves badly. Also
> > similarly, I think that it should be valid to begin a docstring with b"""
> > or u""" (and possibly some combination of b, r, u, before """, although I'm
> > not sure), and these also don't behave correctly.
> >
> >
> >
> > In GNU Emacs 24.5.1 (x86_64-apple-darwin13.4.0, NS apple-appkit-1265.21)
> > of 2015-04-10 on builder10-9.porkrind.org
> > Windowing system distributor `Apple', version 10.3.1348
> > Configured using:
> > `configure --with-ns '--enable-locallisppath=/Library/Application
> > Support/Emacs/${version}/site-lisp:/Library/Application
> > Support/Emacs/site-lisp''
> >
> > Important settings:
> > value of $LANG: C
> > locale-coding-system: utf-8-unix
> >
> > Major mode: Outline
> >
> > Minor modes in effect:
> > goto-address-mode: t
> > TeX-PDF-mode: t
> > tooltip-mode: t
> > electric-indent-mode: t
> > mouse-wheel-mode: t
> > tool-bar-mode: t
> > menu-bar-mode: t
> > file-name-shadow-mode: t
> > global-font-lock-mode: t
> > font-lock-mode: t
> > auto-composition-mode: t
> > auto-encryption-mode: t
> > auto-compression-mode: t
> > buffer-read-only: t
> > line-number-mode: t
> > auto-fill-function: do-auto-fill
> > transient-mark-mode: t
> > view-mode: t
>
> --
> John H. Palmieri
>
prev parent reply other threads:[~2017-08-16 15:05 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-28 19:16 bug#21778: 24.5; python mode: fill-paragraph with raw docstring ignores python-fill-docstring-style John Palmieri
2017-08-16 9:09 ` Charles A. Roelli
[not found] ` <CAHNEuZzTfFHPcuyt8fnzUKmw8f9c=AQiGday7pBP81DmJiwwpw@mail.gmail.com>
2017-08-16 15:05 ` Charles A. Roelli [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m27ey3r1sn.fsf@aurox.ch \
--to=charles@aurox.ch \
--cc=21778-done@debbugs.gnu.org \
--cc=jhpalmieri64@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.