[Rivet] compilation fix

Holger Schulz holger.schulz at durham.ac.uk
Thu 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