You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ko...@apache.org on 2017/08/11 16:18:31 UTC
svn commit: r1804816 - in /subversion/trunk/subversion/libsvn_wc:
adm_crawler.c deprecated.c wc.h
Author: kotkov
Date: Fri Aug 11 16:18:31 2017
New Revision: 1804816
URL: http://svn.apache.org/viewvc?rev=1804816&view=rev
Log:
Make the core of the wc commit do less by moving the logic responsible
for creating temporary files with the detranslated working file contents
to one of its calling sites.
The mentioned core function is svn_wc__internal_transmit_text_deltas(),
and the ability to create such files is only required for the deprecated
svn_wc_transmit_text_deltas2() and svn_wc_transmit_text_deltas() API
functions. That's where we will move this logic.
The side effect of this change is that it also fixes a minor bug — such
temporary files could be left behind in case of some of the errors.
* subversion/libsvn_wc/wc.h
(svn_wc__internal_transmit_text_deltas): Accept a stream to the temporary
file, instead of an out argument where we'd (previously) put the path
to the temporary file.
* subversion/libsvn_wc/adm_crawler.c
(svn_wc__internal_transmit_text_deltas): Don't create and delete the
temporary file, and only optionally wrap the passed-in stream with
the copying_stream() handler.
* subversion/libsvn_wc/deprecated.c
(svn_wc_transmit_text_deltas2): Create the temporary file in this
deprecated function. Pass the disowned tempstream to the core
implementation, svn_wc_transmit_text_deltas2(). Properly close
the stream and don't leave the temporary file on the disk in case
of all errors.
Modified:
subversion/trunk/subversion/libsvn_wc/adm_crawler.c
subversion/trunk/subversion/libsvn_wc/deprecated.c
subversion/trunk/subversion/libsvn_wc/wc.h
Modified: subversion/trunk/subversion/libsvn_wc/adm_crawler.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/adm_crawler.c?rev=1804816&r1=1804815&r2=1804816&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/adm_crawler.c (original)
+++ subversion/trunk/subversion/libsvn_wc/adm_crawler.c Fri Aug 11 16:18:31 2017
@@ -997,7 +997,7 @@ read_and_checksum_pristine_text(svn_stre
svn_error_t *
-svn_wc__internal_transmit_text_deltas(const char **tempfile,
+svn_wc__internal_transmit_text_deltas(svn_stream_t *tempstream,
const svn_checksum_t **new_text_base_md5_checksum,
const svn_checksum_t **new_text_base_sha1_checksum,
svn_wc__db_t *db,
@@ -1027,22 +1027,13 @@ svn_wc__internal_transmit_text_deltas(co
scratch_pool, scratch_pool));
/* If the caller wants a copy of the working file translated to
- * repository-normal form, make the copy by tee-ing the stream and set
- * *TEMPFILE to the path to it. This is only needed for the 1.6 API,
- * 1.7 doesn't set TEMPFILE. Even when using the 1.6 API this file
- * is not used by the functions that would have used it when using
- * the 1.6 code. It's possible that 3rd party users (if there are any)
- * might expect this file to be a text-base. */
- if (tempfile)
- {
- svn_stream_t *tempstream;
-
- /* It can't be the same location as in 1.6 because the admin directory
- no longer exists. */
- SVN_ERR(svn_stream_open_unique(&tempstream, tempfile,
- NULL, svn_io_file_del_none,
- result_pool, scratch_pool));
-
+ * repository-normal form, make the copy by tee-ing the TEMPSTREAM.
+ * This is only needed for the 1.6 API. Even when using the 1.6 API
+ * this temporary file is not used by the functions that would have used
+ * it when using the 1.6 code. It's possible that 3rd party users (if
+ * there are any) might expect this file to be a text-base. */
+ if (tempstream)
+ {
/* Wrap the translated stream with a new stream that writes the
translated contents into the new text base file as we read from it.
Note that the new text base file will be closed when the new stream
@@ -1141,11 +1132,6 @@ svn_wc__internal_transmit_text_deltas(co
investigate. Other commands could be affected,
too, such as `svn diff'. */
- if (tempfile)
- err = svn_error_compose_create(
- err,
- svn_io_remove_file2(*tempfile, TRUE, scratch_pool));
-
err = svn_error_compose_create(
svn_checksum_mismatch_err(expected_md5_checksum, verify_checksum,
scratch_pool,
Modified: subversion/trunk/subversion/libsvn_wc/deprecated.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/deprecated.c?rev=1804816&r1=1804815&r2=1804816&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/deprecated.c (original)
+++ subversion/trunk/subversion/libsvn_wc/deprecated.c Fri Aug 11 16:18:31 2017
@@ -478,20 +478,46 @@ svn_wc_transmit_text_deltas2(const char
const char *local_abspath;
svn_wc_context_t *wc_ctx;
const svn_checksum_t *new_text_base_md5_checksum;
+ svn_stream_t *tempstream;
+ svn_error_t *err;
SVN_ERR(svn_dirent_get_absolute(&local_abspath, path, pool));
SVN_ERR(svn_wc__context_create_with_db(&wc_ctx, NULL /* config */,
svn_wc__adm_get_db(adm_access),
pool));
+ if (tempfile)
+ {
+ /* The temporary file can't be the same location as in 1.6 because the
+ * admin directory no longer exists. */
+ SVN_ERR(svn_stream_open_unique(&tempstream, tempfile,
+ NULL, svn_io_file_del_none,
+ pool, pool));
+ }
+ else
+ {
+ tempstream = NULL;
+ }
- SVN_ERR(svn_wc__internal_transmit_text_deltas(tempfile,
- (digest
- ? &new_text_base_md5_checksum
- : NULL),
- NULL, wc_ctx->db,
- local_abspath, fulltext,
- editor, file_baton,
- pool, pool));
+ err = svn_wc__internal_transmit_text_deltas(svn_stream_disown(tempstream, pool),
+ (digest
+ ? &new_text_base_md5_checksum
+ : NULL),
+ NULL, wc_ctx->db,
+ local_abspath, fulltext,
+ editor, file_baton,
+ pool, pool);
+ if (tempfile)
+ {
+ err = svn_error_compose_create(err, svn_stream_close(tempstream));
+
+ if (err)
+ {
+ err = svn_error_compose_create(
+ err, svn_io_remove_file2(*tempfile, TRUE, pool));
+ }
+ }
+
+ SVN_ERR(err);
if (digest)
memcpy(digest, new_text_base_md5_checksum->digest, APR_MD5_DIGESTSIZE);
Modified: subversion/trunk/subversion/libsvn_wc/wc.h
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/wc.h?rev=1804816&r1=1804815&r2=1804816&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/wc.h (original)
+++ subversion/trunk/subversion/libsvn_wc/wc.h Fri Aug 11 16:18:31 2017
@@ -480,7 +480,7 @@ svn_wc__conflicted_for_update_p(svn_bool
/* Internal version of svn_wc_transmit_text_deltas3(). */
svn_error_t *
-svn_wc__internal_transmit_text_deltas(const char **tempfile,
+svn_wc__internal_transmit_text_deltas(svn_stream_t *tempstream,
const svn_checksum_t **new_text_base_md5_checksum,
const svn_checksum_t **new_text_base_sha1_checksum,
svn_wc__db_t *db,