svn commit: r1139398 - /subversion/trunk/subversion/libsvn_fs_fs/rep-cache.c

[danielsh]

Author: danielsh
Date: Fri Jun 24 18:21:11 2011
New Revision: 1139398

URL: http://svn.apache.org/viewvc?rev=1139398&view=rev
Log:
No functional change.

* subversion/libsvn_fs_fs/rep-cache.c
  (svn_fs_fs__set_rep_reference):
    Tighten variable scope and clarify the 'at this point' reference.

Modified:
    subversion/trunk/subversion/libsvn_fs_fs/rep-cache.c

Modified: subversion/trunk/subversion/libsvn_fs_fs/rep-cache.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/rep-cache.c?rev=1139398&r1=1139397&r2=1139398&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/rep-cache.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/rep-cache.c Fri Jun 24 18:21:11 
2011
@@ -148,7 +148,6 @@ svn_fs_fs__set_rep_reference(svn_fs_t *f
                              apr_pool_t *pool)
 {
   fs_fs_data_t *ffd = fs->fsap_data;
-  representation_t *old_rep;
   svn_sqlite__stmt_t *stmt;
   svn_error_t *err;
 
@@ -173,6 +172,8 @@ svn_fs_fs__set_rep_reference(svn_fs_t *f
   err = svn_sqlite__insert(NULL, stmt);
   if (err)
     {
+      representation_t *old_rep;
+
       if (err->apr_err != SVN_ERR_SQLITE_CONSTRAINT)
         return svn_error_trace(err);
 
@@ -207,10 +208,12 @@ svn_fs_fs__set_rep_reference(svn_fs_t *f
           else
             return SVN_NO_ERROR;
         }
-
-      /* Something really odd at this point, we failed to insert the
-         checksum AND failed to read an existing checksum.  Do we need
-         to flag this? */
+      else
+        {
+          /* Something really odd at this point, we failed to insert the
+             checksum AND failed to read an existing checksum.  Do we need
+             to flag this? */
+        }
     }
 
   return SVN_NO_ERROR;