Save This Code

The following function copies a zero-terminated string pointed by *src into *dest
*/
void strcpy(char *dest, char *src)
{
    while (1)                 // infinite loop begins
    {
        *dest = *src;         // copy one character from *dest to *src
        if (*dest == '')    // if the copied character is zero    
            return;           // abort the loop and return

        ++dest;               // increment destination pointer
        ++src;                // increment source pointer
    }
}

Do you really like this?
Nope not most of it

I specifically asked for Comments and most of what you wrote is not comments on what you try to achieve (important) but explanations of what is going on (if the reader does not know, (s)he should not read in the first place).

copy one character from *dest to *src is not a comment, it an "explanation of compiler behaviour" just like the frequently seen PUSH ACC; save accumulator on stack is explaining an instruction, not its purpuse.

However the function header is mandatory, and in this case, say all that is needed, although I would have liked "standard C library function" added.

Erik