|
[Rivet] compilation fixHolger Schulz holger.schulz at durham.ac.ukThu Sep 15 11:42:27 BST 2016
Hmm there was a recent merge by David, now there is two heads. In order for me not to break anything --- how should we proceed? Holger On 15/09/16 10:46, Andy Buckley wrote: > Apologies, of course it was you that made the commit! Good timing :-) > > > On 15/09/16 10:42, Holger Schulz wrote: >> Hmm funny, I check exactly that in yesterday afternoon. >> >> Holger >> >> >> On 15/09/16 10:27, Andy Buckley wrote: >>> Hasn't this already been done? (By you?) I thought I saw it come >>> through a couple of days ago on the hg commits mailing list... >>> >>> Andy >>> >>> >>> On 15/09/16 08:43, David Grellscheid wrote: >>>> Hi, >>>> >>>> please don't apply this on a C++-98 branch, it'll break on non-clang >>>> setups. On the branches that are -11 by default, use std::tolower. >>>> >>>> David >>>> >>>> >>>> On 15/09/16 06:12, Dmitry Kalinkin wrote: >>>>> Hi again, >>>>> >>>>> Just wanted to check if my email got through. I don't see it on the >>>>> list's archive at >>>>> https://www.hepforge.org/lists-archive/rivet/2016-September/thread.html >>>>> >>>>> >>>>> Cheers, >>>>> Dmitry >>>>> >>>>> On Wed, Sep 7, 2016 at 12:28 AM, Dmitry Kalinkin >>>>> <dmitry.kalinkin at gmail.com> wrote: >>>>>> Dear Rivet developers, >>>>>> >>>>>> The patch below fixes compilation on clang. Hope you will find it >>>>>> useful. >>>>>> >>>>>> Thanks, >>>>>> >>>>>> Dmitry >>>>>> >>>>>> diff --git a/include/Rivet/Tools/Utils.hh >>>>>> b/include/Rivet/Tools/Utils.hh >>>>>> --- a/include/Rivet/Tools/Utils.hh >>>>>> +++ b/include/Rivet/Tools/Utils.hh >>>>>> @@ -101,7 +101,7 @@ namespace Rivet { >>>>>> /// Convert a string to lower-case >>>>>> inline string toLower(const string& s) { >>>>>> string out = s; >>>>>> - std::transform(out.begin(), out.end(), out.begin(), >>>>>> (int(*)(int)) tolower); >>>>>> + std::transform(out.begin(), out.end(), out.begin(), >>>>>> (int(*)(int)) ::tolower); >>>>>> return out; >>>>>> } >>>>>> >>>>>> @@ -109,7 +109,7 @@ namespace Rivet { >>>>>> /// Convert a string to upper-case >>>>>> inline string toUpper(const string& s) { >>>>>> string out = s; >>>>>> - std::transform(out.begin(), out.end(), out.begin(), >>>>>> (int(*)(int)) toupper); >>>>>> + std::transform(out.begin(), out.end(), out.begin(), >>>>>> (int(*)(int)) ::toupper); >>>>>> return out; >>>>>> } >>>>>> >>>>>> >>>>> _______________________________________________ >>>>> Rivet mailing list >>>>> Rivet at projects.hepforge.org >>>>> https://www.hepforge.org/lists/listinfo/rivet >>>>> >>>> _______________________________________________ >>>> Rivet mailing list >>>> Rivet at projects.hepforge.org >>>> https://www.hepforge.org/lists/listinfo/rivet >>>> >>> >>> >> > >
More information about the Rivet mailing list |