fix TStreamerLoop counter offset in member-wise streaming

[tamasgal]

This Pull request:

Fixes TStreamerLoop counter offset in member-wise streaming and adds a regression test.

Changes or fixes:

Checklist:

• tested changes locally
• updated the docs (if necessary)

This PR fixes #22895

[dpiparo]

Thanks for this proposal. Just to double-check: have you had a moment to give a look to ROOT's contributing policy? https://github.com/root-project/root/blob/master/CONTRIBUTING.md#ai-assisted-coding-disclosure

[tamasgal]

no LLM involved, only tooling were clang-format and my ROOT binary reverse engineering skills from the past years ;)

[tamasgal]

many thanks for the comments! Meanwhile me and my colleague (with more that two decades of ROOT experience and the original author of our I/O library) are still doing tests and found other issues, so let me put this on hold for now.

I'll implement your suggestions of course but I fear it's not the whole story ;(

[tamasgal]

as mentioned in the upstream issue, we verified the patch also with the extensive high-level test suite, everything looks fine from our side :)

[dpiparo]

no LLM involved, only tooling were clang-format and my ROOT binary reverse engineering skills from the past years ;)

Thanks for the clarification! I restarted the tests with the newly proposed code.

[github-actions]

Test Results

    22 files      22 suites   3d 19h 51m 15s ⏱️
 3 854 tests  3 854 ✅ 0 💤 0 ❌
75 676 runs  75 676 ✅ 0 💤 0 ❌

Results for commit 600799a.

♻️ This comment has been updated with latest results.

[tamasgal]

could someone please kick off the tests? i messed up the order in the tests CMakeLists but it's fixed now ;)

[dpiparo]

@pcanal, I restarted the ci selecting clean-build to get rid of some artifacts. The PR is ready for review.