> ByteInterleavedRaster is not including the DataBuffer offset in returns from 
> getDataElements
> The super-class sets it in the constructor which runs very much like this 
> subclass except it omits this.
> The parent class of ByteInterleavedRaster is ByteComponentRaster and it uses 
> the DataBuffer offset
> to adjust dataOffsets values used in all calculations.
> 
> Instead ByteInterleavedRaster  does something a bit different than other 
> classes where it includes it in some instance vars
> that also have additional offsets that apply for getPixels and getSamples but 
> aren't used in getDataElements.
> 
> It looks to me as if this is what ByteInterleavedRaster should also do 
> instead.
> All existing tests pass, and this resolves the specific complaint in the bug 
> report.

Phil Race has updated the pull request incrementally with one additional commit 
since the last revision:

  4954405

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/27782/files
  - new: https://git.openjdk.org/jdk/pull/27782/files/24603c3f..081e74de

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=27782&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27782&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/27782.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/27782/head:pull/27782

PR: https://git.openjdk.org/jdk/pull/27782

Reply via email to