Thanks for the feedback everybody!
I uncovered a weird problem when using dialog.py, that is, passing more than 1500 items to tm_dialog through the -p option breaks something. I've hacked together my own tm_dialog.py that uses subprocess to send the plist through stdin, and everything is working again.
I've now released PySmell 0.7.1 which includes this fix plus better error handling and reporting.
Copying the previous email:
I'm very proud to announce the release of PySmell v0.7!
PySmell is an auto-completion library for Python, meant to be plugged in different editors. It uses static analysis to generate a TAGS file for your code, and uses that to give you suggestions. It's very fast - suggestions are instantaneous and analyzing Django 1.0 takes ~15 seconds.
Download from PyPI: http://pypi.python.org/pypi/pysmell/
Issue tracking at Google Code: http://code.google.com/p/pysmell
Development at GitHub: http://github.com/orestis/pysmell/tree/master
The big news for this release is the support for Emacs and TextMate, on top of the already existing Vim support. This isn't tested as rigorously as I'd like, so please report any issues at Google Code.
Thanks, Orestis Markou -- orestis@orestis.gr http://orestis.gr/
On 4 Nov 2008, at 01:44, Orestis Markou wrote:
[...] I uncovered a weird problem when using dialog.py, that is, passing more than 1500 items to tm_dialog through the -p option breaks something. I've hacked together my own tm_dialog.py that uses subprocess to send the plist through stdin, and everything is working again.
I missed you on IRC (CEST) but yes, there is indeed a systme imposed maxium argument size (of about 64 KB), and for that reason, tm_dialog should generally be called with the plist written to its stdin (as you changed it to do).
We’ll gladly patch the default dialog.py with your changes, just send the patch here or paste on IRC, and someone will pick it up :)
Am I correct to assume Python 2.5, seeing that TM2 is going to be Leopard only? Or is this going into a TM1.x release that should be Tiger compatible? I'm afraid I don't have access to Python 2.3...
Anyway, here is the link to the new file. I can't diff it right now because I'm on a Windows machine, unfortunately. I've made some other minor changes as well (documentation and cleanup).
http://github.com/orestis/pysmell/tree/master/pysmell/tm_dialog.py
On Tue, Nov 4, 2008 at 7:42 AM, Allan Odgaard mailinglist@textmate.org wrote:
On 4 Nov 2008, at 01:44, Orestis Markou wrote:
[...] I uncovered a weird problem when using dialog.py, that is, passing more than 1500 items to tm_dialog through the -p option breaks something. I've hacked together my own tm_dialog.py that uses subprocess to send the plist through stdin, and everything is working again.
I missed you on IRC (CEST) but yes, there is indeed a systme imposed maxium argument size (of about 64 KB), and for that reason, tm_dialog should generally be called with the plist written to its stdin (as you changed it to do).
We'll gladly patch the default dialog.py with your changes, just send the patch here or paste on IRC, and someone will pick it up :)
textmate mailing list textmate@lists.macromates.com http://lists.macromates.com/listinfo/textmate
-
Allan Odgaard -
Orestis Markou