CLN: assorted indexing-related cleanups
- _setitem_with_indexer isnt consistent about whether or not it returns anything, make it always-None
- avoid using private loc method from DataFrame

does this change anything user facing? (no?) > does this change anything user facing? (no?)

Correct, nothing user-facing. Thanks!