Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-4783423

DefaultTreeSelectionModel.insureRowContinuity has ambiguous assertion

XMLWordPrintable

    • b25
    • generic, sparc
    • generic, solaris_8

      Name: dsR10051 Date: 11/22/2002


      Javadoc for method of class javax.swing.tree.DefaultTreeSelectionModel
      protected void insureRowContinuity()
      says:
      "
       * @(#)DefaultTreeSelectionModel.java 1.45 02/03/20
          /**
           * Makes sure the currently selected TreePaths are valid
           * for the current selection mode.
           * If the selection mode is <code>CONTIGUOUS_TREE_SELECTION</code>
           * and a RowMapper exists, this will make sure all the rows are
           * contiguous. If the selection isn't contiguous, the selection is
           * reset to contain the first set of contiguous paths.
           * <p>
           * If the selection mode is <code>SINGLE_TREE_SELECTION</code> and
           * more than one TreePath is selected, the selection is reset to
           * contain the first path currently selected.
           */
          protected void insureRowContinuity()
      "
      The assertion "If the selection isn't contiguous, the selection is reset
      to contain the first set of contiguous paths." for CONTIGUOUS_TREE_SELECTION
      selection mode is ambiguous because "first set of contiguous paths"
      has several meanings. (See bug #4485322 for more detatils).
      Evaluation for #4485322 explains algorithm of the selection's retrieval
      if the selection isn't contiguous:
      "
      If the selection isn't contiguous, the selection should be
      reseted to contain the first set of contiguous paths sorted
      in order of rows mapped by RowMapper.
      ###@###.###
      "
      This explanation should be included to javadoc.

      ======================================================================

            sharonz Sharon Zakhour (Inactive)
            sdasunw Sda Sda (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved:
              Imported:
              Indexed: