[Ns-developers] build warning/bug fixes patch

[Mathieu Lacage]

On Wed, 2005-07-06 at 15:30 +0200, Mathieu Lacage wrote:
> On Wed, 2005-07-06 at 09:19 +0200, Mathieu Lacage wrote:
> > This fixes:
> > 1) what looks like a really bad typo in tcp-reno.cc
> > 2) some unsigned/signed confusion and a default statement in the switch
> > cases to quiet the compiler warnings in trace.cc.
> > 3) a small error in agent.cc which will look horribly bad on 64 bit
> > machines.
> > 
> > http://www-sop.inria.fr/dream/personnel/Mathieu.Lacage/ns2/ns-build-
> > warning-1.patch
> 
> 
> The following patches make all of ns build without any warning (except
> for a problem in ip.cc I am working on). 

I will send a patch for this ip.cc thing tomorow.

> http://www-sop.inria.fr/dream/personnel/Mathieu.Lacage/ns2/ns-build-warning-2.patch
> http://www-sop.inria.fr/dream/personnel/Mathieu.Lacage/ns2/ns-build-warning-3.patch
> http://www-sop.inria.fr/dream/personnel/Mathieu.Lacage/ns2/ns-build-warning-4.patch
> http://www-sop.inria.fr/dream/personnel/Mathieu.Lacage/ns2/ns-build-warning-5.patch

Of course, I forgot this one:
http://www-sop.inria.fr/dream/personnel/Mathieu.Lacage/ns2/ns-build-warning-6.patch

> 
> I am going through the test suite and will report on its status tomorow.

validation ok except that it appears the snapshot does not contain some
reference output for some tests which thus fail to pass. For example:

[mlacage at chronos test]$ ./test-all-tcpReset QUIET
no saved test output.  do cvs update -d
or re-run your test with "new" as the last argument.

Is this expected ?

Mathieu
--