Re: [code] [textadept] Textadept 6.5

From: Mitchell <m.att.foicica.com>
Date: Tue, 2 Apr 2013 16:43:44 -0400 (Eastern Daylight Time)

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.

Cheers,
Mitchell

-- 
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 - 16:43:44 EDT

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