Try this one: void strupper(char *str) { for (;*str!='\0';str++) *str=toupper(*str); } ----------------- Jeffrey Hutzelman America Online: JeffreyH11 Internet/BITNET:jh4o+ at andrew.cmu.edu, jhutz at drycas.club.cc.cmu.edu, jh4o at cmuccvma >> Apple // Forever!!! <<