Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Vdebug is currently overwriting the built in 'dir' function when adding itself to the python path. Changing the name of the temporary variable fixes omni completion. #8

Merged
merged 1 commit into from Sep 2, 2012

Conversation

Projects
None yet
2 participants
Contributor

csomme commented Sep 1, 2012

No description provided.

@csomme csomme Python omni completion is currently broken when Vdebug is installed.
Vdebug is currently overwriting the built in 'dir' function when adding
itself to the python path.  Changing the name of the temporary variable
fixes omni completion.
787dc72
Contributor

csomme commented Sep 1, 2012

Hi joonty,

I'm new to github committing to a project on github, I probably should have filed an issue before submitting a pull request?

The plugin is great so far, thank you!

I noticed that once I installed Vdebug I lost python omni completion. I don't know the details of how the omni completion is implemented, but I'm guessing it relies on the 'dir' function, and it was being used as a temporary variable when you were adding the Vdebug modules to sys.path. Changing the name of that variables seems to fix everything.

@joonty joonty added a commit that referenced this pull request Sep 2, 2012

@joonty joonty Merge pull request #8 from csomme/master
Vdebug is currently overwriting the built in 'dir' function when adding itself to the python path.  Changing the name of the temporary variable fixes omni completion.
3ed9194

@joonty joonty merged commit 3ed9194 into joonty:master Sep 2, 2012

Owner

joonty commented Sep 2, 2012

Thanks for that, well spotted. This is the problem with ultra-dynamic languages, you can easily screw stuff up by using the wrong variables names!

I'll package it up as a new version.

@joonty joonty added a commit that referenced this pull request Apr 4, 2013

@joonty joonty Merge pull request #8 from csomme/master
Vdebug is currently overwriting the built in 'dir' function when adding itself to the python path.  Changing the name of the temporary variable fixes omni completion.
bafedaa

@joonty joonty added a commit that referenced this pull request Apr 6, 2013

@joonty joonty Merge pull request #8 from csomme/master
Vdebug is currently overwriting the built in 'dir' function when adding itself to the python path.  Changing the name of the temporary variable fixes omni completion.
a3483bc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment