On Wed, 2008-07-02 at 18:26 +0200, Arfrever Frehtes Taifersar Arahesis
wrote:
> 2008-07-02 18:19:45 Stefan Küng napisa³(a):
> > Hi,
> >
> > In file subversion\libsvn_client\blame.c, line 684 there is an assert:
> >
> > /* The callback has to have been called at least once. */
> > assert(frb.last_filename != NULL);
> >
> > I just got a crash report for TSVN where this assert is being hit: the
> > user passed a revision range which didn't touch the file to blame. So
> > the callback was never hit.
> >
> > I suggest returning an error there, something like "nothing to blame for
> > the specified revision range".
>
> Julian Foad fixed it in r31972.
No I didn't. I just converted all "assert" macros into "SVN_ERR_ASSERT"
macros, which does provide the application with the option of returning
an error object to report the assertion failure instead of aborting, but
doesn't make it right that this supposedly "can't happen" condition in
fact can happen.
- Julian
---------------------------------------------------------------------
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-02 18:36:05 CEST