Upon further investigation, this appears to be a regression.  Rev 7036
<http://bzr.linuxfoundation.org/loggerhead/openprinting/cups-
filters/revision/7036#filter/pdftopdf/pdftopdf_processor.cc> added a
single line: "outputno++;":

      if (param.mirror) {
        curpage->mirror();
      }
      outputno++;  // line added
      proc.add_page(curpage,param.reverse); // reverse -> insert at beginning
    }
  }

 However, looking at the latest revision
<http://bzr.linuxfoundation.org/loggerhead/openprinting/cups-
filters/annotate/head:/filter/pdftopdf/pdftopdf_processor.cc#L299>, that
line appears to have moved:

          if (param.mirror) {
             curpage->mirror();
// TODO? update rect? --- not needed any more
          }
          proc.add_page(curpage,param.reverse); // reverse -> insert at 
beginning
        }
        outputno++;
        curpage=proc.new_page(param.page.width,param.page.height);      
      }

Interestingly, rev 7036 is not listed anywhere on that page.  I'm no
expert, so maybe I'm misinterpreting it, but it suggests to me that
r7036 was effectively lost in a later merge, resulting in the crucial
line moving to the wrong place.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1221520

Title:
  Blank pages inserted in duplex print job

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/cups-filters/+bug/1221520/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to