The unwanted() routine needed to add an explicit sink to certain methods found 
in a block-final Want node.  (Method calls for dispatch:<.=> and Pair.new are 
exempt, however. In the case of .=, it is 'nosink' because it's essentially 
going to cause a side effect anyway, and doing it twice tends to drain Seq 
prematurely if that's what .= returns.  With Pair.new, wrapping in .sink 
suppressed sink warnings on useless use of pairs.)  Other than those 
exceptions, it seems more reasonable to default method calls to sinking on the 
chance that the method will return a sequence that needs to be iterated for its 
side effects.

Fix in 328402599c16077e182bb38baf68e435b8bc1082

Test in b6012e0cd755d9c89ddff00dc45ce81a82bcbdcc

Reply via email to