Harmony
  1. Harmony
  2. HARMONY-43

Performance enhancement for nio buffers bulk get/put methods

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Classlib
    • Labels:
      None

      Description

      The performance of some bulk get/put methods can be enhanced. I will post the patches one by one.

      1. OtherBuffer_patch.txt
        39 kB
        Richard Liang
      2. CharBuffer_patch.txt
        9 kB
        Richard Liang

        Activity

        Hide
        Tim Ellison added a comment -

        Verified by Richard

        Show
        Tim Ellison added a comment - Verified by Richard
        Hide
        Richard Liang added a comment -

        Tim,
        The fix is good. Please close this JIRA. Thanks a lot.

        Show
        Richard Liang added a comment - Tim, The fix is good. Please close this JIRA. Thanks a lot.
        Hide
        Tim Ellison added a comment -

        Richard,

        Thanks for the bulk operation enhancements. They have been applied to the NIO module at repo revision 372498.

        Show
        Tim Ellison added a comment - Richard, Thanks for the bulk operation enhancements. They have been applied to the NIO module at repo revision 372498.
        Hide
        Tim Ellison added a comment -

        Re-opened to capture further changes by Richard.

        Show
        Tim Ellison added a comment - Re-opened to capture further changes by Richard.
        Hide
        Tim Ellison added a comment -

        Thanks Ricahrd, I'll review them now.

        Show
        Tim Ellison added a comment - Thanks Ricahrd, I'll review them now.
        Hide
        Richard Liang added a comment -

        Hello Tim,
        Here is the patch file for all other buffers (except for CharBuffer). The changes are just similar to those of CharBuffer.

        Would you please try to apply the patch? Thanks a lot.

        Show
        Richard Liang added a comment - Hello Tim, Here is the patch file for all other buffers (except for CharBuffer). The changes are just similar to those of CharBuffer. Would you please try to apply the patch? Thanks a lot.
        Hide
        Tim Ellison added a comment -

        Richard,

        Thanks for these enhancements. Committed in NIO module at repo revision 371957.

        Show
        Tim Ellison added a comment - Richard, Thanks for these enhancements. Committed in NIO module at repo revision 371957.
        Hide
        Richard Liang added a comment -

        This patch is created against java.nio which will affect the following files:
        java.nio.CharBuffer
        java.nio.CharArrayBuffer
        java.nio.CharSequenceAdapter
        java.nio.CharToByteBufferAdapter
        java.nio.ReadOnlyCharArrayBuffer
        java.nio.ReadWriteCharArrayBuffer

        Show
        Richard Liang added a comment - This patch is created against java.nio which will affect the following files: java.nio.CharBuffer java.nio.CharArrayBuffer java.nio.CharSequenceAdapter java.nio.CharToByteBufferAdapter java.nio.ReadOnlyCharArrayBuffer java.nio.ReadWriteCharArrayBuffer

          People

          • Assignee:
            Tim Ellison
            Reporter:
            Richard Liang
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development