{"push_id":2832630163,"size":8,"distinct_size":8,"ref":"refs/heads/v6-14-00-patches","head":"3de3f614a9aee11865f46d9e83e22e894a44841b","before":"cfa6e48a41c85b3ed6d902145878a6165b660297","commits":[{"sha":"29d93a67a52a8ca0da8b423e88e37634e2e8b51b","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[DF] Fix ROOT-9542 \"RDataFrame Sum gets confused by std::string\"\n\nROOT::TypeTraits::IsContainer correctly classifies std::string as\nan STL container (since it implements begin(), end(), etc.).\nHowever we do not want to treat std::string as a container of chars\nwhen it comes to RDF actions (i.e. we want to sum/concatenate the\nstrings, not to sum the contents of the strings).\n\nTo this end, two patches have been applied:\n- std::string is now not treated as a container by RDF actions\n- the neutral element of a summation is now assumed to be the\n  default-constructed value if operator- is not present for the\n  type (as it happens in the case of std::string)","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/29d93a67a52a8ca0da8b423e88e37634e2e8b51b"},{"sha":"36dbf213c8dc35453b8024a56e172acba518f2be","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[DF] Add test for Sum of std::strings","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/36dbf213c8dc35453b8024a56e172acba518f2be"},{"sha":"9afd84aadf36a704092ae0e01312f0f0773e16e7","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[VecOps][NFC] Formatting of RAdoptAllocator","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/9afd84aadf36a704092ae0e01312f0f0773e16e7"},{"sha":"6f8eca9675743c0edbf54da82a695853a2e945d4","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[VecOps] Remove redundant RAdoptAllocator::construct overload","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/6f8eca9675743c0edbf54da82a695853a2e945d4"},{"sha":"63b1644d854751c5ab5248f3cd0127c5bce4c07d","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[VecOps][NFC] Add libROOTVecOps as dependency to RAdoptAllocator test","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/63b1644d854751c5ab5248f3cd0127c5bce4c07d"},{"sha":"09d578b089b892d8cf1720df09365ddd30192301","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[VecOps] Specialize RAdoptAllocator<bool>\n\nThe different semantics of std::vector<bool> make memory adoption through a\ncustom allocator more complex -- namely, RAdoptAllocator<bool> must be\nrebindable to RAdoptAllocator<unsigned long>, but if adopted memory is\nreally a buffer of bools, reinterpretation of the adopted buffer as a\ndifferent type is going to break things in horrible ways.\nAs a workaround, RAdoptAllocator<bool> is specialized to be a simple\nallocator that forwards calls to std::allocator, never adopts memory, and\ncan be rebound to any other allocator (as it never adopts memory it can\nrebind the same way that std::allocator can).\n\nThis should resolve ROOT-9453 \"Cannot instantiate RVec<bool>\".","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/09d578b089b892d8cf1720df09365ddd30192301"},{"sha":"ab79bde4e0c0fe96fd9adfde8c21e708286d2725","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[VecOps] Make RAdoptAllocator<T> rebindable to RAdoptAllocator<bool>\n\n...and viceversa. Only non-adopting RAdoptAllocators can be rebound.\nThis makes it possible to copy-construct RVec<bool>'s from other\nRVec<bool>'s.","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/ab79bde4e0c0fe96fd9adfde8c21e708286d2725"},{"sha":"3de3f614a9aee11865f46d9e83e22e894a44841b","author":{"name":"Enrico Guiraud","email":"81a634b61c6957bc62fe9d7d41b6d1b9a925da25@cern.ch"},"message":"[VecOps] Add gtest for ROOT-9453 \"Cannot instantiate RVec<bool>\"","distinct":true,"url":"https://api.github.com/repos/maartenb/cern-root/commits/3de3f614a9aee11865f46d9e83e22e894a44841b"}]}
{"actor":{"display_login":"maartenb"}}