Concept of patch is fine (and we ought to support it), but I dont think this would work as coded (if (tolowerW(c)=='c'||'r') will always be true). Also, can you add a quick test for it (there's cmd.exe /c tests - just add a cmd.exe /r echo test worked).
PS You might want to just hold off until the patch I submitted last night gets committed as well otherwise there will be merge conflicts ("
Change command line parsing away from argv/argc")