Index: www/hacking.html
===================================================================
--- www/hacking.html	(revision 18625)
+++ www/hacking.html	(working copy)
@@ -672,8 +672,8 @@
       then use two underscores following the module prefix.  For
       example:</p>
       <pre>
-         svn_fs_get_rev_prop ()       /* Part of published API. */
-         svn_fs__parse_props ()       /* For internal use only. */
+         svn_fs_get_rev_prop()       /* Part of published API. */
+         svn_fs__parse_props()       /* For internal use only. */
       </pre>
 </li>
 
@@ -779,15 +779,15 @@
        it at the start of each iteration, then destroy it after the loop 
        is done, like so:</p>
        <pre>
-         apr_pool_t *subpool = svn_pool_create (pool);
+         apr_pool_t *subpool = svn_pool_create(pool);
 
          for (i = 0; i &lt; n; ++i)
          {
-           svn_pool_clear (subpool);
-           do_operation (..., subpool);
+           svn_pool_clear(subpool);
+           do_operation(..., subpool);
          }
 
-         svn_pool_destroy (subpool);
+         svn_pool_destroy(subpool);
        </pre>
 </li>
 </ol>
@@ -809,20 +809,20 @@
 code:</p>
 
 <pre>
-      apr_hash_t *persistent_objects = apr_hash_make (pool);
-      apr_pool_t *subpool = svn_pool_create (pool);
+      apr_hash_t *persistent_objects = apr_hash_make(pool);
+      apr_pool_t *subpool = svn_pool_create(pool);
 
       for (i = 0; i &lt; n; ++i)
       {
         const char *intermediate_result;
         const char *key, *val;
         
-        svn_pool_clear (subpool);
-        SVN_ERR (do_something (&amp;intermediate_result, ..., subpool));
-        SVN_ERR (get_result (intermediate_result, &amp;key, &amp;val, ..., pool));
-        apr_hash_set (persistent_objects, key, APR_HASH_KEY_STRING, val);
+        svn_pool_clear(subpool);
+        SVN_ERR(do_something(&amp;intermediate_result, ..., subpool));
+        SVN_ERR(get_result(intermediate_result, &amp;key, &amp;val, ..., pool));
+        apr_hash_set(persistent_objects, key, APR_HASH_KEY_STRING, val);
       }
-      svn_pool_destroy (subpool);
+      svn_pool_destroy(subpool);
 
       return persistent_objects;
 </pre>
@@ -843,8 +843,8 @@
    static foo_t *
    make_foo_object (arg1, arg2, apr_pool_t *pool)
    {
-      apr_pool_t *subpool = svn_pool_create (pool);
-      foo_t *foo = apr_palloc (subpool, sizeof (*foo));
+      apr_pool_t *subpool = svn_pool_create(pool);
+      foo_t *foo = apr_palloc(subpool, sizeof(*foo));
 
       foo-&gt;field1 = arg1;
       foo-&gt;field2 = arg2;
@@ -957,8 +957,8 @@
        something like this:</p>
 
     <pre>
-return svn_error_create (SVN_ERR_FOO, NULL, 
-                         "User not permitted to write file");
+return svn_error_create(SVN_ERR_FOO, NULL, 
+                        "User not permitted to write file");
     </pre>
 
     <p>NOTICE the NULL field... indicating that this error has no
@@ -990,7 +990,7 @@
     <li><p>Throw the error upwards, unmodified:</p>
 
         <pre>
-        error = some_routine (foo);
+        error = some_routine(foo);
         if (error)
           return (error);
         </pre>
@@ -998,7 +998,7 @@
         <p>Actually, a better way to do this would be with the
         SVN_ERR() macro, which does the same thing:</p>
         <pre>
-        SVN_ERR (some_routine (foo));
+        SVN_ERR(some_routine(foo));
         </pre>
      </li>
 
@@ -1006,11 +1006,11 @@
            structure by including it as the "child" argument:</p>
 
         <pre>
-        error = some_routine (foo);
+        error = some_routine(foo);
         if (error)
           {
-           svn_error_t *wrapper = svn_error_create (SVN_ERR_FOO, error,
-                                                    "Authorization failed");
+           svn_error_t *wrapper = svn_error_create(SVN_ERR_FOO, error,
+                                                   "Authorization failed");
            return wrapper;
           }
         </pre>
@@ -1020,11 +1020,11 @@
            your custom message:</p>
 
         <pre>
-        error = some_routine (foo);
+        error = some_routine(foo);
         if (error)
           {
-           return svn_error_quick_wrap (error, 
-                                        "Authorization failed");
+           return svn_error_quick_wrap(error, 
+                                       "Authorization failed");
           }
         </pre>
 
@@ -1032,7 +1032,7 @@
            macro:</p>
 
         <pre>
-          SVN_ERR_W (some_routine (foo), "Authorization failed");
+          SVN_ERR_W(some_routine(foo), "Authorization failed");
         </pre>
     </li>
     </ol>
@@ -1453,7 +1453,7 @@
 <p>And then, in `consume_count' in `cplus-dem.c':</p>
 
 <pre>
-   while (isdigit ((unsigned char)**type))
+   while (isdigit((unsigned char)**type))
      {
        count *= 10;
        count += **type - '0';
@@ -1461,7 +1461,7 @@
          `_Utf390_1__1_9223372036854775807__9223372036854775'
          can cause this function to return a negative value.
          In this case we just consume until the end of the string.  */
-      if (count &gt; strlen (*type))
+      if (count &gt; strlen(*type))
         {
           *type = save;
           return 0;