Minor style point, I like an explicit return None, since functions that don't always have an explicit return sometimes cause bugs. (In this case the calling code expects None, but still)