You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by hw...@apache.org on 2011/05/02 15:43:37 UTC

svn commit: r1098562 - /subversion/trunk/subversion/tests/libsvn_subr/skel-test.c

Author: hwright
Date: Mon May  2 13:43:36 2011
New Revision: 1098562

URL: http://svn.apache.org/viewvc?rev=1098562&view=rev
Log:
* subversion/tests/libsvn_subr/skel-test.c
  (unparse_implicit_length, unparse_list): Avoid alloc'ing never-used values.

Modified:
    subversion/trunk/subversion/tests/libsvn_subr/skel-test.c

Modified: subversion/trunk/subversion/tests/libsvn_subr/skel-test.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/libsvn_subr/skel-test.c?rev=1098562&r1=1098561&r2=1098562&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/libsvn_subr/skel-test.c (original)
+++ subversion/trunk/subversion/tests/libsvn_subr/skel-test.c Mon May  2 13:43:36 2011
@@ -765,11 +765,9 @@ unparse_implicit_length(apr_pool_t *pool
     for (byte = 0; byte < 256; byte++)
       if (skel_is_name( (apr_byte_t)byte))
         {
-          svn_stringbuf_t *str = get_empty_string(pool);
           char buf =  (char)byte;
           svn_skel_t *skel = build_atom(1, &buf, pool);
-
-          str = svn_skel__unparse(skel, pool);
+          svn_stringbuf_t *str = svn_skel__unparse(skel, pool);
 
           if (! (str
                  && str->len == 1
@@ -791,7 +789,7 @@ unparse_list(apr_pool_t *pool)
 {
   /* Make a list of all the single-byte implicit-length atoms.  */
   {
-    svn_stringbuf_t *str = get_empty_string(pool);
+    svn_stringbuf_t *str;
     int byte;
     svn_skel_t *list = empty(pool);
     svn_skel_t *reparsed, *elt;
@@ -839,7 +837,7 @@ unparse_list(apr_pool_t *pool)
 
   /* Make a list of lists.  */
   {
-    svn_stringbuf_t *str = get_empty_string(pool);
+    svn_stringbuf_t *str;
     svn_skel_t *top = empty(pool);
     svn_skel_t *reparsed;
     int i;