Re: [Bug-wget] [Bug-Wget] It's spring cleaning time!

[Darshit Shah]

Here's an updated version of the patch with the C99 comments removed.

It'd be really nice if we can merge this ASAP.
@Tim: Any comments on this patch?


On Mon, May 5, 2014 at 9:44 AM, Darshit Shah <address@hidden> wrote:

>
>
>
> On Mon, May 5, 2014 at 9:40 AM, Tim Ruehsen <address@hidden> wrote:
>
>> On Saturday 03 May 2014 21:18:27 Darshit Shah wrote:
>> > Hello again,
>> >
>> > This took too long. I had the patches, but completely forgot to send
>> them
>> > around. The attached patch was submitted in multiple commits by Tim,
>> >
>> > @Tim: I've taken the liberty to accumulate all the patches into one and
>> > write a cumulative ChangeLog entry. It's HUGE. Do, check it out to see
>> if
>> > it's fine.
>>
>> Hi Darshit,
>>
>> not really HUGE, and those changes are trivial.
>>
>> I found the ChangeLog entry huge. It was so long. It's mostly trivial,
> yes. Wish I had a script to write the ChangeLogs. :)
>
>
>> Could you please change C99 comments into C89 comments in src/http-ntlm.c
>> ?
>> I forgot Wget's C89 subservience... ;-)
>>
> I'll fix that once I'm back on my own machine.
>
>>
>> Thank you.
>>
>> Tim
>>
>
>
>
> --
> Thanking You,
> Darshit Shah
>
>


-- 
Thanking You,
Darshit Shah

0001-Fix-LOTS-of-compiler-warnings.patch
Description: Text Data