-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v626: Emulated Collection Proxy always need the VectorLooper for the StreamerInfoActions #9764
Conversation
…erInfoActions This fixes root-project#9136. Without this commit, SelectLooper would select the 'GenericLooper' in the case of an emulated proxy for STL collection with (in the name) a custom allocator. However the GenericLooper only usable for collection with a compiled collection proxy. In particular, GenericLooper is calling the 'Next' function which is not defined for vector ... and emulated collection. Using it lead to an assert complaining (right fully so) that an 'undefined' function is being called.
Starting build on |
Build failed on ROOT-performance-centos8-multicore/default. Failing tests: |
@phsft-bot build just on ROOT-performance-centos8-multicore/default |
Starting build on |
Build failed on ROOT-performance-centos8-multicore/default. Failing tests: |
@Axel-Naumann The test failure seems both random and unrelated. Merging. |
Starting build on |
Build failed on ROOT-ubuntu2004/soversion. Errors:
|
Build failed on ROOT-ubuntu16/nortcxxmod. Errors:
|
Build failed on mac11/cxx17. Errors:
|
Build failed on ROOT-debian10-i386/cxx14. Errors:
|
Build failed on ROOT-performance-centos8-multicore/default. Errors:
|
Build failed on mac1015/python3. Errors:
|
Build failed on windows10/cxx14. Errors:
|
This fixes #9136.
Without this commit, SelectLooper would select the 'GenericLooper'
in the case of an emulated proxy for STL collection with (in the
name) a custom allocator. However the GenericLooper only usable
for collection with a compiled collection proxy.
In particular, GenericLooper is calling the 'Next' function which
is not defined for vector ... and emulated collection. Using
it lead to an assert complaining (right fully so) that an
'undefined' function is being called.