But if `src_id` is less than 0, it'll fail the `len(sources)` check. `len(sources)` is always >= 0.

I also don't have to agree with everything in a language that I use. ;) I'm more arguing that it's not needed for this comparison. It's overly redundant.