please review the indentation in this function