Patches item #1463133, was opened at 2006-04-02 14:38
Message generated for change (Settings changed) made by russblau
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=603140&aid=146313…
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Closed
Resolution: None
Priority: 5
Private: No
Submitted By: Rich Morin (rich_morin)
Assigned to: Nobody/Anonymous (nobody)
Summary: misc nits
Initial Comment:
General Comments
================
Many of the files to use lines that extend beyond 80
columns. This is awkward for viewing, editing, and
printing.
The readability of the code could benefit from
"Verbose Regular Expressions", which allow white
space, comments, etc. For example, see the RE in
line 533 of wikipedia.py.
http://pywikipediabot.sourceforge.net/
======================================
... get the lastest version ...
latest version
interwiki-graphs/README
=======================
... get the pagacke.
package.
login-data/README
=================
... get the pagacke.
package.
----------------------------------------------------------------------
Comment By: Merlijn S. van Deen (valhallasw)
Date: 2007-07-01 05:14
Message:
Logged In: YES
user_id=687283
Originator: NO
Regular Expressions are write-only anyway. A tool like regexpbuddy (or, is
it exists, its open-source brother) gives a much better insight than
splitting the regexp to multiple lines - and when splitting into multiple
lines, using such a tool gets impossible.
I fixed the spelling in the /README files; 80 characters should be used
but often you'll run into some trouble with several indented blocks. It
does not have a big priority however as most people use an editor that can
handle longer lines.
----------------------------------------------------------------------
Comment By: siebrand (siebrand)
Date: 2007-04-26 15:22
Message:
Logged In: YES
user_id=1107255
Originator: NO
Please let us know if this patch is still applicable to the current code.
If no response is given, the patch will be denied and the issue will be
closed. This message was added in an effort to reduce the number of open
issues on this project. Siebrand
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=603140&aid=146313…