> -----Original Message-----
> From: Paul Burba [mailto:pburba@collab.net]
> Sent: Monday, June 25, 2007 9:26 AM
> To: Lieven Govaerts; Mark Phippard
> Cc: dev@subversion.tigris.org; hwright@tigris.org; Daniel Rall
> Subject: Re: svn commit: r25510 - trunk/subversion/tests/cmdline
<snip>
> Log.c:do_merged_log() - Local variable "svn_boolean_t
> changed" is not initialized before being passed by reference
> to check_history(). Great, except that check_history doesn't
> always modify that arg:
>
> /* Set INFO->HIST to the next history for the path *if* there
> is history
> * available and INFO->HISTORY_REV is equal to or greater
> than CURRENT.
> *
> * *CHANGED is set to TRUE if the path has history in the
> CURRENT revision,
> >>>> * otherwise it is not touched. <<<<
> *
> * If we do need to get the next history revision for the path, call
> * get_history to do it -- see it for details.
> */
Hyrum,
Just thinking out loud, but is it really wise to have functions like
check_history() that take a predicate argument but only sometimes
modifies it? Doesn't seem we do that anywhere else that I can recall
(though I haven't made an exhaustive search).
Paul
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 25 15:45:05 2007