Thanks. Good feedback. I think making the first syntax change here makes perfect sense and that's much better than what I had, but I don't think I want to make a local method wrapper for `validate_db_name`.

It actually kind of annoys me that we created a wrapper for `ok?` here. It might make the code a little prettier in a few places and save a few trivial keystrokes, but adds unnecessary method lookup overhead. I don't really see the value.

For `ok?` it could be argued that it's widespread use makes it useful to wrap locally, but that's not the case for `validate_db_name`. In the case of `validate_db_name` specifically, this is literally the only place in the entire driver we're actually using it. 

It should actually just be a private method here in `Mongo::DB`. I'm not sure why the `validate_db_name` method was put in `Mongo::Support` in the first place and I really wanted to move it. :-/

Unfortunately, as is too often the case, this method in `Mongo::Support` isn't private and I can't move it since it might break someone who may be using it externally.