From: <bc...@us...> - 2016-12-22 20:32:43
|
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 |