Minor on style, let us char* instead of char *. Same for the rest of the file with pointers.