remove exhaustive list and add 'default: break;' to switch statement
initialize p as 'p = s.data();'