Exuberant Ctags / Bugs / #207 Python variables stating with "def" are shown as functions

#207 Python variables stating with "def" are shown as functions

Status: closed-fixed

Owner: Elias Pschernig

Labels: None

Priority: 5

Updated: 2008-03-29

Created: 2007-12-22

Creator: Andrey Nordin

Private: No

If the name of a Python variable starts with "def" it is marked as a function.

I'm using exuberant-ctags 5.7 on Ubuntu 7.10 taken from its main repo. I've attached a trivial example: there is a variable named "default" inside a function "foo". Both the function and the variable (its name is parsed as "ault") are tagged by ctags.

Discussion

Andrey Nordin - 2007-12-22

Conflict of "def" and "default" in Python

exuberant-ctags_python_def_bug.py

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andrey Nordin - 2007-12-22

summary: Variables stating with "def" are shown as functions --> Python variables stating with "def" are shown as functions
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elliott Hughes - 2007-12-23

Logged In: YES
user_id=1127237
Originator: NO

assigning to Python maintainer.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elliott Hughes - 2007-12-23

assigned_to: nobody --> elias
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elias Pschernig - 2007-12-23

Logged In: YES
user_id=32894
Originator: NO

Thanks, fixed in revision 632.

Btw., why did this not appear on the mailing list? I thought bug reports are supposed to get forwarded there. Or did it appear, and my spam filter ate it?

Also.. I forgot how the regression testing works, so just committed without :/ Link to refresh my memory would be appreciated though :)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elias Pschernig - 2007-12-23

status: open --> open-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elliott Hughes - 2007-12-23

Logged In: YES
user_id=1127237
Originator: NO

> why did this not appear on the mailing list?

i don't think i've ever seen that work.

> Link to refresh my memory would be appreciated though :)

i've added the details to MAINTAINERS. it's too complicated for anyone to remember, and too hard to find.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elliott Hughes - 2008-02-16

Logged In: YES
user_id=1127237
Originator: NO

this particular fix has been merged downstream in Ubuntu, and they've requested that Debian merge it. i know there haven't been many changes since 5.7, but it might still be worth putting out a point release to save all the distros the hassle?

when sourceforge sends me the notification of this comment, i'll forward it to darren to see what he says.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elliott Hughes - 2008-02-25

Logged In: YES
user_id=1127237
Originator: NO

i've added a test case.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elliott Hughes - 2008-03-29

Logged In: YES
user_id=1127237
Originator: NO

(closing because a test case has been added and the NEWS file has been updated.)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Elliott Hughes - 2008-03-29

status: open-fixed --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.