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=1463133...
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=1463133...