fix TypedArray set

[sablib]

It should follow the spec and behave the same with TypedArray in Typescript.

Re: #

Changelog

---

Continuous Integration

This pull request:

• needs automatic test cases check.

  Manual trigger with @cocos-robot run test cases afterward.

• does not change any runtime related code or build configuration

  If any reviewer thinks the CI checks are needed, please uncheck this option, then close and reopen the issue.

---

Compatibility Check

This pull request:

• changes public API, and have ensured backward compatibility with deprecated features.
• affects platform compatibility, e.g. system version, browser version, platform sdk version, platform toolchain, language version, hardware compatibility etc.
• affects file structure of the build package or build configuration which requires user project upgrade.
• introduces breaking changes, please list all changes, affected features and the scope of violation.

[github-actions]

✅ Package size is not changed

Interface Check Report

This pull request does not change any public interfaces !

[dumganhar]

Good catch.
In this case, I think another two set methods should also consider the current _byteOffset while copying data.

template <typename T>
template <typename SrcType>
typename std::enable_if_t<std::is_same<T, SrcType>::value, void> TypedArrayTemp<T>::set(const TypedArrayTemp<SrcType> &array, uint32_t offset) {
    CC_ASSERT(_buffer);
    uint32_t dstByteOffset = offset * BYTES_PER_ELEMENT;
    uint32_t srcByteOffset = array.byteOffset();
    uint32_t srcCount = array.length();
    CC_ASSERT(dstByteOffset + srcCount * TypedArrayTemp<SrcType>::BYTES_PER_ELEMENT <= _byteEndPos);
    memcpy(_buffer->_data + dstByteOffset, array._buffer->_data + srcByteOffset, array.byteLength());
}
template <typename T>
template <typename SrcType>
typename std::enable_if_t<!std::is_same<T, SrcType>::value, void> TypedArrayTemp<T>::set(const TypedArrayTemp<SrcType> &array, uint32_t offset) {
    CC_ASSERT(_buffer);
    uint32_t dstByteOffset = offset * BYTES_PER_ELEMENT;
    uint32_t srcByteOffset = array.byteOffset();
    uint32_t srcCount = array.length();
    uint32_t remainCount = (_byteEndPos - dstByteOffset) / BYTES_PER_ELEMENT;
    CC_ASSERT_LE(srcCount, remainCount);
    for (uint32_t i = 0; i < srcCount; ++i) {
        (*this)[offset + i] = reinterpret_cast<T>(array[i]);
    }
}

[sablib]

Add a commit to fix these function templates.

[dumganhar]

@cocos-robot run test cases

[dumganhar]

@cocos-robot run test cases

[dumganhar]

@cocos-robot run test cases

[github-actions]

@sablib, Please check the result of run test cases:

• Test Platform: PR-Test
• Editor Version: v3.8.5
• Task Result: PASS
• Task URL: https://cctest.cocos.org/#/reportview/PR-TEST/761c2b2c-6cb5-423a-93ac-0e9e2299f132/-1
• GitHub Action: https://github.com/cocos/cocos-engine/actions/runs/11771381393

Task Details

[github-actions]

@sablib, Please check the result of run test cases:

• Test Platform: PR-Test
• Editor Version: v3.8.5
• Task Result: PASS
• Task URL: https://cctest.cocos.org/#/reportview/PR-TEST/d8056e0e-38b6-484e-9ba3-8e902aebffb2/-1
• GitHub Action: https://github.com/cocos/cocos-engine/actions/runs/11771381393

Task Details