i'd also put it inside an if, in case it fails...
if(mail()){ // mail succeeded} else { // mail failed}