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 | d16b233ed8a81860e706a008bde82dfa568d4dcd (patch) | |
| tree | cddea1a36d6b88042ae21a39031325c0bc28cb6e /generic | |
| parent | e22aa5854dc50ba15a3966093bec99d37059b93e (diff) | |
| download | tcl-d16b233ed8a81860e706a008bde82dfa568d4dcd.zip tcl-d16b233ed8a81860e706a008bde82dfa568d4dcd.tar.gz tcl-d16b233ed8a81860e706a008bde82dfa568d4dcd.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)) { |