Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(579)

Issue 492523002: Fixed command line parting when no output is needed. (Closed)

Created:
6 years, 4 months ago by Vitaly Buka (NO REVIEWS)
Modified:
6 years, 4 months ago
Reviewers:
Bo Xu, inferno
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Visibility:
Public.

Description

Fixed command line parting when no output is needed. R=bo_xu@foxitsoftware.com Committed: https://pdfium.googlesource.com/pdfium/+/8cf33bc

Patch Set 1 : Tue Aug 19 14:14:39 PDT 2014 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M samples/pdfium_test.cc View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Vitaly Buka (NO REVIEWS)
6 years, 4 months ago (2014-08-19 21:14:54 UTC) #1
Bo Xu
On 2014/08/19 21:14:54, Vitaly Buka wrote: lgtm
6 years, 4 months ago (2014-08-19 21:20:04 UTC) #2
Vitaly Buka (NO REVIEWS)
Committed patchset #1 manually as 8cf33bc (presubmit successful).
6 years, 4 months ago (2014-08-19 21:22:24 UTC) #3
Bo Xu
6 years, 4 months ago (2014-08-19 21:22:47 UTC) #4
Message was sent while issue was closed.
On 2014/08/19 21:20:04, Bo Xu wrote:
> On 2014/08/19 21:14:54, Vitaly Buka wrote:
> 
> lgtm

I am thinking, if some one write: pdfium_test --ppm --bmp --emf a.pdf, shall we
output all three types of output files?

Powered by Google App Engine
This is Rietveld 408576698