ICEpdf
  1. ICEpdf
  2. PDF-152

CLONE -minor bug in PrintServices example

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1.2
    • Fix Version/s: 4.0
    • Component/s: Examples
    • Labels:
      None
    • Environment:
      LINUX: 2.6.31-15-generic #50-Ubuntu SMP Tue Nov 10 14:54:29 UTC 2009 i686 GNU/Linux
      Java(TM) SE Runtime Environment (build 1.6.0_16-b01)
    • Workaround Exists:
      Yes
    • Workaround Description:
      modify example as described above

      Description

      On Line 97 of PrintServices.java there is a for loop which lists the users printers:

      for (int i = 0, max = services.length - 1; i < max; i++) {

      If the user has only a single printer (as I do on my Ubuntu system) then no printers are listed.

      Instead of testing for "i < max" the test should be "i <= max"

      for (int i = 0, max = services.length - 1; i <= max; i++) {

      At the same time for clarity it may also be worth changing the user prompt on line 99 to include "printer number" :

                          "Please select the printer number your wish to print to (q to quit):");

      Regards

      Alec Cormack

      p.s. ICEpdf is great software. Many thanks!

        Activity

        Hide
        Alec Cormack added a comment -

        Sorry. Opened this by mistake, Please close. see PDF-153 for issue

        Show
        Alec Cormack added a comment - Sorry. Opened this by mistake, Please close. see PDF-153 for issue

          People

          • Assignee:
            Patrick Corless
            Reporter:
            Alec Cormack
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: