Hey all, Yesterday, I filed a ticket with Infra to alter our Reply-To. Looks like it has gone through.
The Reply-To *only* includes dev now, rather than both dev/commits. That should make it much easier for us to keep discussion off the commits list. Cheers, -g ---------- Forwarded message ---------- From: <julianf...@apache.org> Date: Thu, Mar 11, 2010 at 11:13 Subject: svn commit: r921916 - in /subversion/trunk/subversion: libsvn_repos/fs-wrap.c libsvn_repos/load.c svn/status-cmd.c To: comm...@subversion.apache.org Author: julianfoad Date: Thu Mar 11 16:13:39 2010 New Revision: 921916 URL: http://svn.apache.org/viewvc?rev=921916&view=rev Log: Eliminate or comment a few type casts. * subversion/libsvn_repos/fs-wrap.c (svn_repos_fs_get_mergeinfo): Comment a type cast. * subversion/libsvn_repos/load.c (renumber_mergeinfo_revs): Eliminate two type casts. * subversion/svn/status-cmd.c (print_status): Eliminate a type cast. Modified: subversion/trunk/subversion/libsvn_repos/fs-wrap.c subversion/trunk/subversion/libsvn_repos/load.c subversion/trunk/subversion/svn/status-cmd.c Modified: subversion/trunk/subversion/libsvn_repos/fs-wrap.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_repos/fs-wrap.c?rev=921916&r1=921915&r2=921916&view=diff ============================================================================== --- subversion/trunk/subversion/libsvn_repos/fs-wrap.c (original) +++ subversion/trunk/subversion/libsvn_repos/fs-wrap.c Thu Mar 11 16:13:39 2010 @@ -586,6 +586,8 @@ svn_repos_fs_get_mergeinfo(svn_mergeinfo void *authz_read_baton, apr_pool_t *pool) { + /* Here we cast away 'const', but won't try to write through this pointer + * without first allocating a new array. */ apr_array_header_t *readable_paths = (apr_array_header_t *) paths; svn_fs_root_t *root; apr_pool_t *iterpool = svn_pool_create(pool); Modified: subversion/trunk/subversion/libsvn_repos/load.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_repos/load.c?rev=921916&r1=921915&r2=921916&view=diff ============================================================================== --- subversion/trunk/subversion/libsvn_repos/load.c (original) +++ subversion/trunk/subversion/libsvn_repos/load.c Thu Mar 11 16:13:39 2010 @@ -279,8 +279,8 @@ renumber_mergeinfo_revs(svn_string_t **f void *val; apr_hash_this(hi, &key, NULL, &val); - merge_source = (const char *) key; - rangelist = (apr_array_header_t *) val; + merge_source = key; + rangelist = val; /* Possibly renumber revisions in merge source's rangelist. */ for (i = 0; i < rangelist->nelts; i++) Modified: subversion/trunk/subversion/svn/status-cmd.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svn/status-cmd.c?rev=921916&r1=921915&r2=921916&view=diff ============================================================================== --- subversion/trunk/subversion/svn/status-cmd.c (original) +++ subversion/trunk/subversion/svn/status-cmd.c Thu Mar 11 16:13:39 2010 @@ -180,7 +180,7 @@ print_status(void *baton, scache->path = apr_pstrdup(sb->cl_pool, path); scache->status = svn_wc_dup_status2(status, sb->cl_pool); - path_array = (apr_array_header_t *) + path_array = apr_hash_get(sb->cached_changelists, cl_key, APR_HASH_KEY_STRING); if (path_array == NULL) {
