User talk:SqrtNegInf: Difference between revisions
→Range consolidation
Thundergnat (talk | contribs) (→Range consolidation: new section) |
SqrtNegInf (talk | contribs) |
||
Line 43:
Doh. Good catch on the Range consolidation task. I cut out a bunch of logic dealing with excluded boundaries and completely missed the the enclosed range case. To be fair, there weren't any test cases that exercised that or I probably would have noticed. Thanks. --[[User:Thundergnat|Thundergnat]] ([[User talk:Thundergnat|talk]]) 23:30, 5 February 2019 (UTC)
: Actually, there is: <tt>[-6, -1], [-4, -5]</tt>, otherwise I doubt I would have found the problem. I don't know if Paddy3118 was clever or lucky in using that pairing. Your output as shown was correct, so I figured an earlier version worked, but the urge to stream-line the code caught up with you...
| |||