diff options
| author | dgp <dgp@users.sourceforge.net> | 2013-03-19 14:26:10 (GMT) |
|---|---|---|
| committer | dgp <dgp@users.sourceforge.net> | 2013-03-19 14:26:10 (GMT) |
| commit | a515c79f52c3d0927a7377700467074b80b8d169 (patch) | |
| tree | cddea1a36d6b88042ae21a39031325c0bc28cb6e /generic | |
| parent | f02a6bd750b0c1da6b070dc9faf45562fa1c560d (diff) | |
| download | tcl-a515c79f52c3d0927a7377700467074b80b8d169.zip tcl-a515c79f52c3d0927a7377700467074b80b8d169.tar.gz tcl-a515c79f52c3d0927a7377700467074b80b8d169.tar.bz2 | |
3597000 Consistent [file copy] result.
Diffstat (limited to 'generic')
| -rw-r--r-- | generic/tclFCmd.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/generic/tclFCmd.c b/generic/tclFCmd.c index 5ad7063..382e9f3 100644 --- a/generic/tclFCmd.c +++ b/generic/tclFCmd.c @@ -691,15 +691,14 @@ CopyRenameOneFile(interp, source, target, copyFlag, force) * so it should be quite clear */ errfile = target; - /* - * We now need to reset the result, because the above call, - * if it failed, may have put an error message in place. - * (Ideally we would prefer not to pass an interpreter in - * above, but the channel IO code used by - * TclCrossFilesystemCopy currently requires one) - */ - Tcl_ResetResult(interp); } + /* + * We now need to reset the result, because the above call, + * may have left set it. (Ideally we would prefer not to pass + * an interpreter in above, but the channel IO code used by + * TclCrossFilesystemCopy currently requires one) + */ + Tcl_ResetResult(interp); } if ((copyFlag == 0) && (result == TCL_OK)) { if (S_ISDIR(sourceStatBuf.st_mode)) { |