[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-commits
Subject:    Re: svn commit: r1889114 - /subversion/trunk/subversion/tests/libsvn_subr/task-test.c
From:       Nathan Hartman <hartman.nathan () gmail ! com>
Date:       2021-04-23 5:16:39
Message-ID: CAJT2EHoh-F1rpG75rEaipGUqX1SykJUN6Gig3vEWjQ=fjEKoLA () mail ! gmail ! com
[Download RAW message or body]

On Fri, Apr 23, 2021 at 12:59 AM <hartmannathan@apache.org> wrote:
> Modified: subversion/trunk/subversion/tests/libsvn_subr/task-test.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/libsvn_subr/task-test.c?rev=1889114&r1=1889113&r2=1889114&view=diff
>  ==============================================================================
> --- subversion/trunk/subversion/tests/libsvn_subr/task-test.c (original)
> +++ subversion/trunk/subversion/tests/libsvn_subr/task-test.c Fri Apr 23 04:59:29 \
> 2021 @@ -124,32 +124,32 @@ counter_func(void **result,
> 
> if (value > 1)
> {
> -      partial_result = apr_palloc(result_pool, sizeof(partial_result));
> +      partial_result = apr_palloc(result_pool, sizeof(*partial_result));
> *partial_result = 1;
> value -= *partial_result;
> 
> sub_task_pool = svn_task__create_process_pool(task);
> 
> -      partial_baton = apr_palloc(sub_task_pool, sizeof(partial_baton));
> +      partial_baton = apr_palloc(sub_task_pool, sizeof(*partial_baton));
> *partial_baton = MAX(1, value / 2);
> value -= *partial_baton;
> 
> -      SVN_ERR(svn_task__add_similar(task, sub_task_pool,
> +      SVN_ERR(svn_task__add_similar(task, sub_task_pool,
> partial_result, partial_baton));
> }
> 
> if (cancel_func)
> SVN_ERR(cancel_func(cancel_baton));
> -
> +

My editor removed some trailing whitespaces, such as the example
above, and I forgot to un-remove them before committing. :-/

Nathan


[prev in list] [next in list] [prev in thread] [next in thread]