|
From: Hans D. <dul...@eg...> - 2000-09-18 16:52:55
|
"Frank V. Castellucci" wrote: > > Hans Dulimarta wrote: > > > > > > Yes, doxygen-1.2.1-20000829. > > > > I just checked in my fix to bugs 113982, 113983, and 113985. > > The ex22 sample code now runs ok. The source of these bugs > > were found > > in EventSemaphoreGroup and EventSemaphore. In ESG, counter > > in theUsedMap > > was decremented twice causing premature call to delete > > operator. > > > > In ES, the bug was subtle. If you remember in our exchange > > of e-mails > > in Auguest, I initially proposed to implement "hold" > > operation as SetValue (1), > > "wait" as Z(s), and "trigger/signal" operation as P(s), but > > then I changed it to: > > "hold" as setValue (-1) and trigger as V(s). > > > > Apparently, setValue (-1) is not a valid semctl SETVAL > > operation. So I reverted the > > implementation to my initial approach [setValue (1) and > > trigger as P(s)]. > > > > Now for bug 113984: you mention that the indentation level > > is 4, but I think that > > C++ coding standard mentions an indentation level of 3. > > You are correct, I think I was refering to the level of > indentation...will have to look at the original defect. > > Great to hear you got those more important ones though. What do you use > for a debugger? I was using DDD with appropriate gdb level for thread > handling but I was never able to break-point on a threads execution. > I was having the same problem too. So, I gave up using ddd and just used the most traditional debugger in the universe, the 'cerr<<' "debugger" :-). When fixing those three bugs, I had to put several 'cerr <<' in other files too: AbstractSemaphore.cpp, SemaphoreCommon.cpp in order to find out that SETVAL -1 wasthe intervening operation. > > -- > Frank V. Castellucci > http://corelinux.sourceforge.net > OOA/OOD/C++ Standards and Guidelines for Linux > http://PythPat.sourceforge.net > Pythons Pattern Package > _______________________________________________ > Corelinux-develop mailing list > Cor...@li... > http://lists.sourceforge.net/mailman/listinfo/corelinux-develop -- Hans Dulimarta, Ph.D. dul...@co... P: 517-432-7589 http://www.egr.msu.edu/~dulimart F: 760-281-7691 http://corelinux.sourceforge.net Elec. & Comp. Engg., Mich. State Univ., E. Lansing, MI 48824 |