[RFC] Return bool from find_win_for_buf

Saw a comment about this in my last pull. Turns out the function is only called in two places.
> Perhaps we should add a note in our style guide

I've been considering asking about whether `!= NULL` for pointers is really necessary as well.