[Ns-developers] ns-3.6 Daily Bug Status for Wednesday, September 30, 2009

[Mathieu Lacage]

On Wed, 2009-09-30 at 22:52 -0700, Tom Henderson wrote:

> > 1.Bug 407 nor P1 Linu tomh at tomh.org NEW OLSR is missing HNA support 
> 
> Suggest to drop the above to P2-- it is a feature add.

+1 for P2.

> 
> > 4.Bug 555 nor P1 All ns-bugs at isi.edu NEW DCF immediate access bug 
> 
> The above has been hanging around for a while-- is it at a deadlock?

I don't really know what to do about this bug. I see the bug, we have a
testcase, but the proposed fix is making the code even more scary than
it already is (makes a function have side-effects, overall confusion
about function names: what does it mean to have a backoff expired ? how
different is it from backoffslot == 0 ?). It's not clear if there is a
better way to fix the bug. To summarize, I still strongly dislike the
proposed fix: I can't make myself commit it.

> > 6.Bug 612 nor P1 All ns-bugs at isi.edu NEW example scripts
> 
> The above should not block a release, IMO.  It probably could be easily 
> disposed of if there was agreement/support for adding some 
> subdirectories into examples directory.

We need to make building examples optional: build times are getting out
of hand for daily work. Gustavo: do you think you could propose a patch
to do something like that ?

> 
> > 8.Bug 645 nor P1 All ns-bugs at isi.edu NEW patch fixes for opening stats file
> > with OMNeT++ 
> 
> I think you can assign the above to Joe, who has already responded in 
> the tracker.  However, given that stats is in src/contrib, and that 
> there appears to need to be more work done to fix this formatting issue, 
> I would advocate bumping it to P2.

+1 for P2.

> 
> > 10.Bug 648 nor P1 All ns-bugs at isi.edu NEW Missing Doxygen for Several
> > Helpers 
> 
> If we have the extra week, the above may be fixable.  I will take it 
> once my other P1 bugs are done, if no one beats me to it.

I have done 2, there are 2 left:
tap-bridge-helper.h
ipv4-static-routing-helper.h

> > 15.Bug 669 nor P1 All ns-bugs at isi.edu NEW main-packet-printer broken 
> 
> I will take this, with the caveat that if we run out of time, this is 
> marked a WONTFIX and removed from the samples/ directory.  It is a 
> sample that uses the old packet API, and has already been removed from 
> samples/wscript for some time.

There is already good sample code to do this in src/common/packet.cc in
the Packet::Print method so, I would suggest to kill
samples/main-packet-printer.cc.

Mathieu