Merge lp:~mnordhoff/loggerhead/333797-debug-print into lp:loggerhead

Proposed by Matt Nordhoff
Status: Merged
Merged at revision: not available
Proposed branch: lp:~mnordhoff/loggerhead/333797-debug-print
Merge into: lp:loggerhead
Diff against target: None lines
To merge this branch: bzr merge lp:~mnordhoff/loggerhead/333797-debug-print
Reviewer Review Type Date Requested Status
Michael Hudson-Doyle Approve
Review via email: mp+4261@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Matt Nordhoff (mnordhoff) wrote :

This removes a print statement that causes log output like this:

DEBUG:paste.httpserver.ThreadPool:Added task (0 tasks queued)
INFO:loggerhead./trunk:Getting information for RevisionUI: 0.1818850040435791 secs
INFO:loggerhead:127.0.0.1 - - [07/Mar/2009:07:33:45 +0000] "GET /trunk/revision/295/loggerhead/history.py HTTP/1.1" 200 - "http://localhost:8080/trunk/revision/295" "Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.7) Gecko/2009021906 Firefox/3.0.7"
loggerhead/controllers/annotate_ui.py loggerhead/history.py False
loggerhead/history.py loggerhead/history.py True
INFO:loggerhead./trunk:Rendering RevisionUI: 0.034359216690063477 secs, 8851 bytes

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

yes yes, now land it yourself :)

review: Approve

Updating diff...

An updated diff will be available in a few minutes. Reload to see the changes.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'loggerhead/templatefunctions.py'
2--- loggerhead/templatefunctions.py 2009-03-05 22:56:59 +0000
3+++ loggerhead/templatefunctions.py 2009-03-07 07:29:26 +0000
4@@ -42,7 +42,6 @@
5 if link_style == 'fragment':
6 def file_link(filename):
7 if currently_showing:
8- print filename, currently_showing, filename == currently_showing
9 if filename == currently_showing:
10 return '<b><a href="#%s">%s</a></b>' % (
11 cgi.escape(filename), cgi.escape(filename))

Subscribers

People subscribed via source and target branches