static char temp [256];

1. Sein dämliches C-Beispiel ist net Locale-Aware. toupper() [ANSI C] hätt' ihm die Hacke erspart und könnte je nach Encoding deutsche Umlaute, Kyrillischen Text umwandeln, ... und wär' einfacher gewesen. Wahrscheinlich hat er's extra so gemacht, damit's in Pascal einfacher aussieht obwohl man in Wirklichkeit _WEIT_ mehr Aufwand investieren müßte.
   
2. sein temp[i]=0 hätt' er sich sparen können, indem er die Schleife anders baut.
   
3. return temp; ist nun aber wirklich net nur ein Schönheitsfehler sondern _BUG_. Niemals ein return aus lokalen Variablen. Er hat sich mit einem static geholfen, damit das trotzdem klappt - nur verhaut er damit den Stack und hält sinnlos Speicher belegt. Kurzum: Er hat's _net_ überrissen.
   
4. 
   

char *mein_upper(char *s)
{
        char *p, *temp;
        p=temp=strdup(s);

        while ( *p != '\0' ) {
                *p=toupper(*p);
                p++;
        }
        return temp;
}