Re: "url = url" ? LLVM complains.

Post the patch file here.

Eliezer

On 12/20/2012 8:41 PM, Tianyin Xu wrote:
> Thanks, Alex.
> OK, I'll do it. So I post it on bugzilla, right?
>
> Best,
>
> On Thu, Dec 20, 2012 at 10:19 AM, Alex Rousskov
> <rousskov_at_measurement-factory.com> wrote:
>> On 12/20/2012 10:57 AM, Tianyin Xu wrote:
>>> Yes, that's what I did. There're 7 places in Squid-3.2.5 having a=a
>>> lines. Thanks for the confirmation!
>>
>> Please consider posting a patch so that we do not miss any a=a lines
>> (since we cannot reproduce the issue specific to your compiler).
>>
>>
>> Thank you,
>>
>> Alex.
>>
>>
>>> On Thu, Dec 20, 2012 at 9:53 AM, Alex Rousskov
>>> <rousskov_at_measurement-factory.com> wrote:
>>>> On 12/20/2012 10:44 AM, Tianyin Xu wrote:
>>>>> On Thu, Dec 20, 2012 at 9:37 AM, Alex Rousskov
>>>>> <rousskov_at_measurement-factory.com> wrote:
>>>>>> On 12/20/2012 10:25 AM, Tianyin Xu wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> I'm using LLVM to compile the squid source code. LLVM stopped at the
>>>>>>> statement "a = a" and regarded as an error.
>>>>>>>
>>>>>>> For example, in squid-3.2.5/libltdl/loaders/preopen.c, there're a
>>>>>>> couple of such statements:
>>>>>>>
>>>>>>> 112 url = url;
>>>>>>> 113 request = request;
>>>>>>> 114 msgid = msgid;
>>>>>>
>>>>>> I do not see such statements in my copy of libltdl/loaders/preopen.c.
>>>>>> Please note that preopen.c is not Squid code. It is libltdl code.
>>>>>>
>>>>>
>>>>> Osh, I'm very sorry, Alex. It's in
>>>>> helpers/external_acl/kerberos_ldap_group/support_ldap.cc (I did
>>>>> something wrong with the clipboard) It's in the
>>>>> squid-3.2.5-20121217-r11739.tar.gz tarball.
>>>>
>>>> It looks like a bug to me (with no runtime effects), but I do not know
>>>> much about that code. Try removing all those a=a lines. Normally, they
>>>> do not do anything anyway.
>>>>
>>>>
>>>> HTH,
>>>>
>>>> Alex.
>>>>
>>>
>>>
>>>
>>
>
>
>

-- 
Eliezer Croitoru
https://www1.ngtech.co.il
sip:ngtech_at_sip2sip.info
IT consulting for Nonprofit organizations
eliezer <at> ngtech.co.il