You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2010/08/14 14:43:55 UTC

svn commit: r985482 - /subversion/branches/performance/subversion/libsvn_subr/io.c

Author: stefan2
Date: Sat Aug 14 12:43:55 2010
New Revision: 985482

URL: http://svn.apache.org/viewvc?rev=985482&view=rev
Log:
File I/O functions tend to be called frequently and many of them call the
do_io_file_wrapper_cleanup() function. Most of the times, the function code
will be almost a no-op making the call itself expensive. Thus, we suggest
to the compiler to inline this function.

* subversion/libsvn_subr/io.c
  (do_io_file_wrapper_cleanup): inline

Modified:
    subversion/branches/performance/subversion/libsvn_subr/io.c

Modified: subversion/branches/performance/subversion/libsvn_subr/io.c
URL: http://svn.apache.org/viewvc/subversion/branches/performance/subversion/libsvn_subr/io.c?rev=985482&r1=985481&r2=985482&view=diff
==============================================================================
--- subversion/branches/performance/subversion/libsvn_subr/io.c (original)
+++ subversion/branches/performance/subversion/libsvn_subr/io.c Sat Aug 14 12:43:55 2010
@@ -1710,7 +1710,7 @@ svn_io_file_lock2(const char *lock_file,
 
 /* Data consistency/coherency operations. */
 
-static svn_error_t *
+static APR_INLINE svn_error_t *
 do_io_file_wrapper_cleanup(apr_file_t *file, apr_status_t status,
                            const char *msg, const char *msg_no_name,
                            apr_pool_t *pool);
@@ -2809,7 +2809,7 @@ svn_io_file_open(apr_file_t **new_file, 
 }
 
 
-static svn_error_t *
+static APR_INLINE svn_error_t *
 do_io_file_wrapper_cleanup(apr_file_t *file, apr_status_t status,
                            const char *msg, const char *msg_no_name,
                            apr_pool_t *pool)