Re: [code] [textadept] Textadept 6.5

From: Ryan Pusztai <rpusztai.att.gmail.com>
Date: Tue, 2 Apr 2013 22:08:36 -0400

Hi Mitchell,

On Tue, Apr 2, 2013 at 4:43 PM, Mitchell <m.att.foicica.com> wrote:

> Hi Ryan,
>
>
> On Tue, 2 Apr 2013, Ryan Pusztai wrote:
>
> Hi Mitchell,
>>
>> On Tue, Apr 2, 2013 at 12:37 AM, Mitchell <m.att.foicica.com> wrote:
>>
>> Hi Ryan,
>>>
>>> I know you mentioned that the paths accepted are now just strings, but
>>>
>>>> during my conversion I left it as a table and the error is a bit vague
>>>> (/bin/textadept_6.5.x86_64/****core/file_io.lua:433: attempt to call
>>>> method
>>>>
>>>> 'gmatch' (a nil value)). Maybe check that the type is table? Just a
>>>> thought.
>>>>
>>>>
>>> I would prefer to keep the API simple. If you still want to use a table,
>>> my recommendation is to call `io.snapopen(table.concat(t, '\n'))`. This
>>> is
>>> a valid concern though, so thanks for bringing it up.
>>>
>>
>>
>> It is much more about the error message not being clear than about using a
>> table. What about making the error clearer?
>>
>
> The problem is that an error message would require new localizations and
> that effort doesn't seem worth it for this particular API, so I won't be
> working on it.

Ok just thought I would mention it.

--
Regards,
Ryan
-- 
You are subscribed to code.att.foicica.com.
To change subscription settings, send an e-mail to code+help.att.foicica.com.
To unsubscribe, send an e-mail to code+unsubscribe.att.foicica.com.
Received on Tue 02 Apr 2013 - 22:08:36 EDT

This archive was generated by hypermail 2.2.0 : Wed 03 Apr 2013 - 06:38:41 EDT