[bind10-dev] whether to merge incomplete stuff

Shane Kerr shane at isc.org
Tue Mar 1 10:43:58 UTC 2011


Jinmei,

On Fri, 2011-02-25 at 12:35 -0800, JINMEI Tatuya / 神明達哉 wrote:
> Via cppcheck reports (in branch trac613) I noticed a few incomplete
> (actually "empty") methods have been merged.  These are, specifically,
> as follows:
>  - MessageCache::dump()
>  - MessageCache::load()
>  - MessageCache::resize()
>  - RRsetCache::dump()
>  - RRsetCache::load()
>  - RRsetCache::resize()

<snip/>

> So, my suggestion is to remove the declaration and (empty) definition
> of these methods, and just note a possibility of these features as a
> future \todo item in comments.
> 
> Does that make sense?

I totally agree.

--
Shane




More information about the bind10-dev mailing list