Hi Joel, very nice!!

Didn't find any problems this time... checked with docstrings, kwargs, defaults, end of doc, etc.
The only observation is that code-analysis complained about an unused variable -- the first sCurrentLine = oSelection.getCursorLineContents(), in the run method, as you make the same assignment some lines below before using it again.

So, what do you (all) think about which would be the best way to make those scripts available... I was thinking something as putting them (with the authors permission) toghether with the pydev distribution...
The authors name would then appear on the script itself and in a brief help in the homepage toghether with an explanation on how the script works.

Joel: what do you think about showing as the first script in that list? ;-)

Cheers,

Fabio

On 4/19/06, Joel Hedlund <[EMAIL PROTECTED]> wrote:
Hi!

New version of my script here. This time I feel it's good enough to call an
alpha (comments still appreciated, of course).

> I installed and played a little with your script - looking good.

Thanks!

> Since I am congenitally unable to accept any compiler error messages

:-)

>     document = document [EMAIL PROTECTED]

If I used a global variable named 'document' in my previous version it was most
likely because of involuntary namespace pollution, and I hope the new version
is free of it.

Thanks for your time checking my script out!
/Joel Hedlund



Reply via email to