"Chia-liang Kao" <clkao_at_clkao.org> writes:
>>> - PATH is the path being unlocked, USERNAME is the person doing it. */
>>> + PATH is the path being unlocked, USERNAME is the person doing it,
>>> + TOKEN is the lock token to be unlocked, and BREAK-LOCK is a flag if
>>> + the user is breaking the lock. */
>>> svn_error_t *
>>> svn_repos__hooks_pre_unlock(svn_repos_t *repos,
>>
>> Likewise for TOKEN.
>
> Actually token should never be null, so the code for checking null is
> probably redundant.
Oh, I'm not talking about the code, I'm talking about the doc string.
If TOKEN should never be null, then that needs to be documented.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-07-21 22:57:05 CEST