Merge lp://qastaging/~kroq-gar78/simple-scan/fix-765780 into lp://qastaging/~simple-scan-team/simple-scan/trunk
Proposed by
Aditya V
Status: | Merged |
---|---|
Merged at revision: | 573 |
Proposed branch: | lp://qastaging/~kroq-gar78/simple-scan/fix-765780 |
Merge into: | lp://qastaging/~simple-scan-team/simple-scan/trunk |
Diff against target: |
25 lines (+5/-3) 1 file modified
src/book.vala (+5/-3) |
To merge this branch: | bzr merge lp://qastaging/~kroq-gar78/simple-scan/fix-765780 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Robert Ancell | Pending | ||
Aditya V | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2012-03-29.
Description of the change
Add the page number to the first file in a multi-file scan, and start counting from one for multi-file scans.
To post a comment you must log in.
Hi,
Thanks very much for that! The functionality of the patch is correct, but do you mind changing the way it works? It's a little bit complicated the way it is.
Instead of having the separate loop when get_n_pages() == 1 make make_indexed_file() take a new argument 'n_pages'. If n_pages == 0 then don't modify the name, otherwise put the numeric suffix on it.
Also note the pages should start at one as requested by Michael in the bug report.