nit: just use `fail!("invalid border style")`. Then you don't need the useless `false`.