[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [Nbd] [PATCH] Document format of strings in one place, limit to 4096 bytes



On 6 Apr 2016, at 22:51, Wouter Verhelst <w@...112...> wrote:

> On Wed, Apr 06, 2016 at 10:19:21PM +0100, Alex Bligh wrote:
>> On 6 Apr 2016, at 21:27, Wouter Verhelst <w@...112...> wrote:
>>> Also, most of those strings exist solely to make user interaction
>>> easier. In that context, a 4K string is *way* too long; I'm thinking
>>> something like 256 seems saner (80 characters, times three because
>>> UTF-8, round up to the next power of two).
>> 
>> I agree it's way too long (partly for the above reason). I'm happy
>> to go with 256, but it's conceivable we might want to reconsider
>> if it's a 'MUST'.
> 
> We could do SHOULD for 256 and MUST for 4K; I'd be okay with that (4K is
> ridiculously large enough that it shouldn't happen in practice, anyway)

Fine by me.

-- 
Alex Bligh







Reply to: