-
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
io: Allow implicit conversion from vector<pair<A<B> > to map<A,B> #17591
Conversation
Test Results 18 files 18 suites 4d 21h 26m 29s ⏱️ Results for commit 4d6101a. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Regarding the roottest PR: shouldn't it just test this conversion?
For once, it was the reverse and this PR is fixing an issue that was (re)discovered by the test reorganization and subsequent enabling of a test that had been forgotten. |
The Windows failure is 'real' and was not noticed before as the test was not run on Windows (due to being part of a gmake tests) |
ba9c270
to
dde6d28
Compare
This avoid the risk of applying the dereference twice.
dde6d28
to
9f64056
Compare
This reverts commit 459246d.
Both this function expect the value of a data member which is a pointer rather than its address: ``` TString TBufferJSON::JsonWriteMember(const void *ptr, TDataMember *member, TClass *memberClass, Int_t arraylen) TString TBufferJSON::ConvertToJSON(const void *ptr, TDataMember *member, Int_t compact, Int_t arraylen) ```
There was a timeout in |
The roottest companion PR is root-project/roottest#1255