Re: Test framework; new counter class

[Federico Montesino Pouzols]

        Well, both things -the unified counter template and the
CppUnit tests- are very interesting.

        One point that you should take into account for the atomic
counter stuff is that we removed the use of asm/atomic.h (actually
there should not be any references to it left) a few weeks ago. Look
for a bug report that we received from the debian maintainer of
commoncpp2, at the September and October archives of bug-commoncpp. I
think we should not wait to add the counter template to HEAD, and if
you think the interface is stable, it should also get into RELEASE1.

        If there are no more last minute updates or suggestions for
the CppUnit part of the patch for now, I would add it to CVS. Or do
you, Albert and Chad, prefer to merge or compare your tests before?

On Tue, Dec 31, 2002 at 01:10:10PM +0200, Albert Strasheim wrote:
> Hello,
> 
> On Tue, 31 Dec 2002, Albert Strasheim wrote:
> 
> > Attached is a patch that contains the following:
> 
> New patch attached. diff'd against the same CVS snapshot.
>  
> > - some basic code for a test framework for Common C++ based on the 
> > CppUnit test library,
> > 
> > - a counter class (called UberCounter for now) meant to unify the 
> > Counter, MutexCounter and AtomicCounter classes,
> 
> I've cleaned this up quite a bit. I've added reasonably complete doxygen 
> documentation, and moved a bunch of methods that were public but are 
> supposed to be protected.
> 
> For some reason, doxygen isn't convinced that UberCounter is in the 
> Common C++ namespace, though. Also, some @c's aren't resolving...
>  
> > - minor tweaks to configure.in and some Makefile.am files related to 
> > gcc compiler flags.
> > 
> > The patch is against commoncpp2 in CVS on Tue Dec 31 02:00:00 GMT+2 2002 
> > or thereabouts.
> 
> The rest of the stuff is still there. I don't have the energy to split 
> up the patch into the various possibilities. :-)
> 
> Cheers,
> 
> Albert


> _______________________________________________
> Bug-commoncpp mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/bug-commoncpp