Re: [Etherape-devel] [etherape:etherape] New commit [a2d78a] by R.Ghetta

[Zev W. <ze...@be...>]

On Thu, Dec 22, 2016 at 09:04:56PM +0100, bc...@us... wrote:
>On 22/12/2016 12:55, Zev Weiss wrote:
>>Hi Riccardo,
>>In commit 6ffe90 I added some logic to lift the arbitrary 10-column
>>limit on the column layout code, but this commit seems to have
>>re-introduced it (around line 394 or so of src/main.c, in the function
>>parse_position_line()).  Is there some particular reason that was
>>needed, or would it be OK to take the '> 10' check back out?
>>
>>Thanks,
>>Zev
>Hmm, perhaps no.
>There was a small problem with node_matches_spec_list crashing when 
>the column list had holes and at first I blamed the missing <10, then 
>forgot to remove the check after fixing the real bug.
>
>Ciao,
>Riccardo

Okay, thanks -- now back to unlimited in the latest commit.

Zev