This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add locale/charset support to mkshortcut


On 15 August 2010 22:43, Charles Wilson wrote:
> On 8/15/2010 5:26 PM, Andy Koppe wrote:
>> On 15 August 2010 22:11, Charles Wilson wrote:
>>> So, if you want to forward port your locale/charset changes and
>>> cleanup-handling changes for mkshortcut (probably as two separate
>>> patches) I'll try to act on them more rapidly this time. :-)
>>
>> So you'd be ok with not freeing stuff before exit()? That would make
>> things quite a bit easier.
>
> I'd rather have both "proper(?)" handling of allocated memory AND
> locale/charset support -- but if it's a choice between one and the
> other, I pick locale support. ÂIn fact, I'd rather take your changes as
> they are (well, without the now-gratuitous signature changes to
> license(), help(), usage(), and version()) and then later worry about my
> personal foibles vis handling of allocated memory -- especially as
> free-before-exit is really quite pedantic.
>
>> (Belated apologies for being too strident
>> about this back then.)
>
> Ah, it's only code.
>
> I'm going to go ahead and roll out 1.4.4 today with the return-value
> fix, and if I need to spin 1.4.6 later this week with your new stuff
> that's ok.

Great! I'll try to get to it this week.

Andy


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]