Re: Test framework; new counter class

[David Sugar]

I am a little hesitent in the idea of putting something new or different into 
the r1 branch, especially since I think this new ubercounter should actually 
depreciate the existing counter stuff.  I do not know if anyone uses existing 
counters but unless it were a true drop in replacement, there is concern will 
break behavior or interfaces existing apps depend on, and I do not want 
people to start coding for "ubercounter" if later we make "ubercounter" 
simply "counter".

On Tuesday 31 December 2002 16:39, Federico Montesino Pouzols wrote:
> 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
>
> _______________________________________________
> Bug-commoncpp mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/bug-commoncpp