Other than @mikepb and @jmendeth's suggestions I like it, and don't see anything immediately wrong with it.