bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#58023: 29.0.50; end-of-defun in python-mode does not work with neste


From: kobarity
Subject: bug#58023: 29.0.50; end-of-defun in python-mode does not work with nested function with multi-line arguments
Date: Fri, 23 Sep 2022 21:01:20 +0900
User-agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (Gojō) APEL-LB/10.8 EasyPG/1.0.0 Emacs/29.0.50 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO)

Hi,

The following steps will place the point at the end of the class C,
not the end of the method a.

1. emacs -Q
2. Load the following Python file using M-x find-file

#+begin_src python
class C:
    def a(self
          ):
        pass

    def b(self):
        pass
#+end_src

3. C-n (M-x forward-line)
4. C-M-e (M-x end-of-defun)

This happens when the method (nested function) has multi-line
arguments.

This is caused by the wrong behavior of
`python-nav-beginning-of-defun'.  When the point is inside of the
parens of the nested function, it fails to detect the beginning of the
nested function and moves the point to the beginning of the outer
class/function.

The root cause of this problem exists in
`python-info-looking-at-beginning-of-defun'.  It checks syntax context
to detect correct keywords, but it checks at the current position
although keywords are searched for at the beginning of the line.  If
the point is inside of the parens of the function definition, syntax
context is "paren" and `python-info-looking-at-beginning-of-defun'
returns nil, which is not expected.  The syntax context should be
checked after moving the point to the beginning of the line.

Attached is a patch to fix this issue.

Best Regards,

--

In GNU Emacs 29.0.50 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo
 version 1.16.0, Xaw scroll bars) of 2022-09-23 built on ubuntu
Repository revision: 8238d87e15e2b2b730b6290965d14787650a15af
Repository branch: master
System Description: Ubuntu 22.04.1 LTS

Configured using:
 'configure --without-libgmp'

Configured features:
ACL CAIRO FREETYPE GIF GLIB GNUTLS GSETTINGS HARFBUZZ JPEG LIBSELINUX
LIBXML2 MODULES NOTIFY INOTIFY PDUMPER PNG SECCOMP SOUND SQLITE3 THREADS
TIFF TOOLKIT_SCROLL_BARS X11 XDBE XIM XPM LUCID ZLIB

Important settings:
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  show-paren-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
  blink-cursor-mode: t
  line-number-mode: t
  indent-tabs-mode: t
  transient-mark-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t

Load-path shadows:
None found.

Features:
(shadow sort mail-extr emacsbug message mailcap yank-media puny dired
dired-loaddefs rfc822 mml mml-sec password-cache epa derived epg rfc6068
epg-config gnus-util text-property-search time-date subr-x mm-decode
mm-bodies mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader
cl-loaddefs cl-lib sendmail rfc2047 rfc2045 ietf-drums mm-util
mail-prsvr mail-utils term/screen term/xterm xterm byte-opt gv bytecomp
byte-compile cconv rmc iso-transl tooltip eldoc paren electric uniquify
ediff-hook vc-hooks lisp-float-type elisp-mode mwheel term/x-win x-win
term/common-win x-dnd tool-bar dnd fontset image regexp-opt fringe
tabulated-list replace newcomment text-mode lisp-mode prog-mode register
page tab-bar menu-bar rfn-eshadow isearch easymenu timer select
scroll-bar mouse jit-lock font-lock syntax font-core term/tty-colors
frame minibuffer nadvice seq simple cl-generic indonesian philippine
cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese composite emoji-zwj charscript
charprop case-table epa-hook jka-cmpr-hook help abbrev obarray oclosure
cl-preloaded button loaddefs faces cus-face macroexp files window
text-properties overlay sha1 md5 base64 format env code-pages mule
custom widget keymap hashtable-print-readable backquote threads inotify
dynamic-setting system-font-setting font-render-setting cairo x-toolkit
x multi-tty make-network-process emacs)

Memory information:
((conses 16 45893 5646)
 (symbols 48 5782 0)
 (strings 32 15032 1212)
 (string-bytes 1 425800)
 (vectors 16 8334)
 (vector-slots 8 104450 6830)
 (floats 8 24 229)
 (intervals 56 200 0)
 (buffers 1000 10))

Attachment: 0001-Fix-syntax-check-in-python-info-looking-at-beginning.patch
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]