[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: CVS update: subversion/subversion/libsvn_fs txn-table.c

From: Jim Blandy <jimb_at_zwingli.cygnus.com>
Date: 2000-12-14 13:44:14 CET

> Actually, isn't this a facet of all calls to svn_fs__*_to_dbt()? That you're
> feeding data to the DBT for insertion, and that nothing will attempt to
> change the DBT after that point?
>
> IOW, why wouldn't this cast move inside of str_to_dbt and a "const" applied
> to the str_to_dbt prototype?

The other *_to_dbt calls actually copy their arguments, so making
their arguments const is appropriate there. str_to_dbt points to its
argument, and we can't promise that the DB won't modify it (there are
some calls that both read and write a DBT). So the cast needs to be
made by code which can be sure it's correct.
Received on Sat Oct 21 14:36:17 2006

This is an archived mail posted to the Subversion Dev mailing list.