Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in

Issue 5973044: Directs pnmtopng stderr output to dev/null (Closed)

Can't Edit
Can't Publish+Mail
Start Review
7 years, 2 months ago by PhilEHolmes
7 years, 2 months ago


I'm directing this to dev null because there's no point in using a logfile here. The message from pnmtopng is: pnmtopng: 256 colors found and it displays this for greyscale images. If this line fails, it will be pngtopnm that will fail, and we will continue to get the error messages from that. I can see no circumstance when pnmtopng will fail, and so we don't need to see its output.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M Documentation/ly-examples/GNUmakefile View 1 chunk +1 line, -1 line 0 comments Download


Total messages: 2
Please review.
7 years, 2 months ago (2012-04-01 16:51:40 UTC) #1
Graham Percival
7 years, 2 months ago (2012-04-01 17:53:30 UTC) #2
Sure, go ahead and push.

Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b