910 not raised on conditional checkpoint in infinite loop

[jakkdl]

# this raises 910
async def no_checkpoint():
    if ...:
        await trio.sleep("1")
# and this
async def no_checkpoint():
    while bar():
        if ...:
            await trio.sleep("1")
# but not this
async def no_checkpoint():
    while True:
        if ...:
            await trio.sleep("1")

same for try/except.

The problem likely stems from the fact that the function never returns, and as such it never returns without checkpointing - which is where 910 would trigger. Found it while exploring #227, and would be particularly problematic if/when expanding ASYNC100 to use Visitor91x.

[jakkdl]

A possible solution is adding a dedicated rule for "[infinite] loop without checkpoint in loop body", see #227 (comment)