>>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:

Angus> Content-Type: text/plain Content-Transfer-Encoding: 8bit

Angus> The attached patch was created with cvs diff -p -u >
Angus> angus.patch bzip2 angus.patch

Angus> It allows the head branch of CVS to be compiled with DEC cxx
Angus> and DEC's implementation of std::string. (Does little more than
Angus> replace string::clear() with string::erase().)

I am a bit surprised by the following:
-Bullet::Bullet(const int f, const int c, const int s)
+Bullet::Bullet(int f, int c, int s)

Wouldn't it be better to fix the header file?

Also, this
-// need to make the c++ compiler fint the correct version of abs.
+// need to make the c++ compiler find the correct version of abs.
 // This is at least true for g++.
+#ifdef __GNUG__
 using std::abs;
+#endif
is not right, since one should never condition on compilers (against
autoconf philosophy). The best is probably to add the proper
definitions in cheaders/cmath.

I also tried to find in the list archive which string method of gcc
2.x.y was supposed to have arguments in the wrong order, and could not
find it. Lars, any clue?

JMarc

Reply via email to